RFR: 8159745: Remove java.net.Parts

Chris Hegarty chris.hegarty at oracle.com
Fri Jun 17 14:26:28 UTC 2016



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.


More information about the net-dev mailing list