On Apr 6, 2017, at 11:07 AM, Chris Hegarty <chris.hega...@oracle.com> wrote:

>> On 6 Apr 2017, at 17:50, Michael McMahon <michael.x.mcma...@oracle.com> 
>> wrote:
>> 
>> Looks fine to me Chris. Stylistically, the boolean tests
>> in line 104 could remove the == true obviously, but not a big deal.
> 
> Thanks, I’ll make this change before pushing.

+1

Thanks,

Brian

Reply via email to