Review Request: Warnings cleanup in sun.management and its subpackages

Kurchi Hazra kurchi.subhra.hazra at oracle.com
Mon Dec 5 11:49:21 PST 2011



On 12/5/2011 8:44 AM, Chris Hegarty wrote:
>
> Some trivial comments:
>   ConnectorAddressLink.java
>     L172 - L178, you can use 'for(Counter c: counters)' and remove
>     subsequent redundant cases.
>   MappedMXBeanType.java
>     L76, cast to (Class<?) ???

Thanks! I changed these.

>
> Some of the public classes have API changes. Are these classes in any 
> way supported? Some have Javadoc comments that would make me think 
> this, but maybe not given the name space.

I tried being careful - but I guess I did touch public classes. Maybe I 
will wait for Mandy's review and go over these again, and then send out 
a final webrev.


- Kurchi


>
> -Chris.
>
> On 02/12/2011 23:09, Kurchi Hazra wrote:
>> Hi,
>>
>> http://cr.openjdk.java.net/~khazra/7117570/webrev.00/
>>
>> Please review this webrev that cleans warnings in sun.management and its
>> subpackages. These packages are now warning-free I believe.
>>
>> Looking forward to your comments. I have used the serialver tool to
>> generate all the serial version UIDs that I have added.
>>
>>
>>
>> Thanks,
>>
>> Kurchi
>>

-- 
-Kurchi



More information about the jdk8-dev mailing list