On Jan 14, 2015, at 5:31 PM, Chris Hegarty <chris.hega...@oracle.com> wrote:

> On 14/01/15 16:18, Mandy Chung wrote:
>> On 1/14/2015 8:04 AM, Chris Hegarty wrote:
>>> 
>>> http://cr.openjdk.java.net/~chegar/8061297/webrev.01/webrev/
>> 
>> Looks okay to me.
>> 
>> In CallerSensitiveFinder.java line 57, 80, 137 - not sure if you were
>> thinking to pass "pool" to the CallerSensitiveFinder constructor. Looks
>> like some editing you meant to cleanup.
>> 
>>   nit: line 76 a space between "classes" and "=="
>>   line 207 can be removed.
> 
> D'oh! Fixed both of these. Webrev updated in place.
> 

Looks good.

Paul.

Reply via email to