RFR: 8159745: Remove java.net.Parts

Claes Redestad claes.redestad at oracle.com
Fri Jun 17 14:23:25 UTC 2016



On 2016-06-17 16:26, Chris Hegarty wrote:
>
>
> 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.

Thanks for the reviews, pushed!

/Claes


More information about the net-dev mailing list