On 2017-08-30 16:10, Alan Bateman wrote:
On 29/08/2017 21:26, Claes Redestad wrote:
Webrev: http://cr.openjdk.java.net/~redestad/8186930/jdk.00/
This looks good to me.

Thanks!

I just wonder if ParseUtil should keep the lowMask/highMask methods in comments so that further maintainers can satisfy themselves that the values are correct.

I thought leaving a comment in ParseUtil that they can be still found in URI was sufficient (cuts back on the redundancy), but I can keep them in comments in both places if you think that's preferable.

/Claes

Reply via email to