[OpenJDK 2D-Dev] JDK 9: RFR[2]: 8033716: Fix raw and unchecked lint warnings in com.sun.imageio

Henry Jen henry.jen at oracle.com
Thu Feb 20 21:01:09 UTC 2014


Hi,

Please review the latest update, I think this should address the issues 
raised,

http://cr.openjdk.java.net/~henryjen/jdk9/8033716/2/webrev/

- revert the clone method changes so that return type remains Object
- break long lines
- fix fallthrough warnings as Andrew suggested.

Cheers,
Henry


On 02/20/2014 10:09 AM, Phil Race wrote:
> On 2/20/2014 12:05 AM, Henry Jen wrote:
>> On 02/19/2014 02:59 PM, Henry Jen wrote:
>>> On 02/19/2014 01:46 PM, Phil Race wrote:
>>>>
>>>> W.r.t the following change ...
>>>>
>>>> http://cr.openjdk.java.net/~henryjen/jdk9/8033716/1/webrev/src/share/classes/com/sun/imageio/plugins/jpeg/DHTMarkerSegment.java.sdiff.html
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> 145     class Htable implements Cloneable {
>>>>
>>>> ...
>>>> <208          protected Object clone() {
>>>>
>>>>>  208         protected Htable clone()
>>>>
>>>> ---------
>>>>
>>>> exactly what warning is this suppressing ?
>>>
>>> This eliminate "unchecked" cast warning when calling this method to get
>>> an instance with correct class type.
>>>
>>
>> That's not exactly correct, it just eliminate the need to cast, not
>> warning.
>>
>> Would you advice to revert them or keep it?
>
> In that case, I'd suggest to revert them
>
> -phil.



More information about the 2d-dev mailing list