OK unless there are objections from Sergey tomorrow I will plan to push it 
as-is.

Thanks,

Brian

On Aug 23, 2016, at 5:12 PM, Philip Race <philip.r...@oracle.com> wrote:

> Still fine with me.
> 
> -phil.
> 
> On 8/23/16, 4:18 PM, Brian Burkhalter wrote:
>> 
>> Hi Sergey,
>> 
>> I have added some comments in both the source and the test:
>> 
>> http://cr.openjdk.java.net/~bpb/8149562/webrev.03/
>> 
>> Please indicate whether you think this is sufficient. The CCC request has 
>> been approved therefore pushing this code is pending only review approval.
>> 
>> Thanks,
>> 
>> Brian
>> 
>> On Aug 23, 2016, at 11:45 AM, Sergey Bylokhov <sergey.bylok...@oracle.com> 
>> wrote:
>> 
>>> On 23.08.16 21:41, Sergey Bylokhov wrote:
>>>> Is it possible, just for the record, to provide a comment why NPE was
>>>> changed to IAE.
>>> I meant the first check node==null.
>>> 
>>> actually this code looks a little bit strange:
>>>> 493         } catch (NullPointerException npe) {
>>>> 494             throw new IllegalArgumentException(npe);
>>>> 495         }
>> 

Reply via email to