On 17/06/16 15:05, Claes Redestad wrote:


On 2016-06-17 16:04, Alan Bateman wrote:
On 17/06/2016 14:43, Claes Redestad wrote:

Hi,

URL.java defines a package-private class Parts, which can be
trivially inlined in the one place where it's currently used.

Webrev: http://cr.openjdk.java.net/~redestad/8159745/webrev.01/
Bug: https://bugs.openjdk.java.net/browse/JDK-8159745

I suspect that class had other usages in old releases. Your patch
looks okay, maybe change `ind` to `index` or something that doesn't
look like `int` :-)

Thanks, I'll update ind before push. :-)

+1  Thanks Claes.

-Chris.

Reply via email to