<AWT Dev> [8] Review request for 8022184 - JDK8 java.awt Static and Raw type warnings fix

srikalyan chandrashekar srikalyan.chandrashekar at oracle.com
Tue Aug 27 00:02:02 PDT 2013


Artem these explicit casts existed and is now redundant because the 
warnings are fixed
For Ex: 1.Class<SomeClass> obj = ....
             2.obj = cl.newInstance();

Explicit cast is not required in the line 2 because type is now known 
due to fix. There could potentially be many such do you want them to be 
identified and fixed as part of this effort? The thing is we need to 
look around warning areas a little bit..

--
Thanks
kalyan

On 8/26/13 6:39 AM, Artem Ananiev wrote:
>
> On 8/23/2013 9:24 PM, srikalyan chandrashekar wrote:
>> Antony, Thanks for the review. Here's the renewed link
>> <https://github.com/srikalyc/JDKfixes/blob/master/java.awt.static_raw_webrev_new.zip> 
>>
>> covering the gaps.
>
> Here is the updated link at cr.openjdk.java.net:
>
> http://cr.openjdk.java.net/~art/srikalyc/8022184.01/
>
> A few comments:
>
> 1. GraphicsEnvironment.java:109 - class cast is redundant
>
> 2. KeyboardFocusManager.java:2655 - class cast is redundant
>
> 3. KeyboardFocusManager.java:2985 - class cast is redundant
>
> 4. DefaultKeyboardFocusManager.java:816 - class cast is redundant
>
> 5. DefaultKeyboardFocusManager.java:1067 - class cast is redundant
>
> Thanks,
>
> Artem
>
>> -- 
>> Thanks
>> kalyan
>>
>> On 8/23/13 3:44 AM, Anthony Petrov wrote:
>>> Hi Srikalyan,
>>>
>>> Thanks for fixing the warnings. A couple of comments:
>>>
>>> src/share/classes/java/awt/KeyboardFocusManager.java
>>>> 3065             proxyActive = (Field)
>>>> AccessController.doPrivileged(new PrivilegedAction<Field>() {
>>>
>>> I suppose an explicit type-cast may now be removed from here.
>>>
>>>
>>> src/share/classes/java/awt/WaitDispatchSupport.java
>>>>  227 AccessController.doPrivileged(new PrivilegedAction<Object>() {
>>>
>>> Usually Void is used instead of Object if we know that we don't use
>>> the return value.
>>>
>>> Otherwise the fix looks good to me.
>>>
>>> -- 
>>> best regards,
>>> Anthony
>>>
>>> On 08/22/2013 08:40 PM, Artem Ananiev wrote:
>>>>
>>>> On 8/22/2013 8:25 PM, srikalyan chandrashekar wrote:
>>>>> Hi team ,  could someone review the fix
>>>>>      Bug      : https://jbs.oracle.com/bugs/browse/JDK-8022184
>>>>
>>>> Here is the right link visible for everyone:
>>>>
>>>> http://bugs.sun.com/view_bug.do?bug_id=8022184
>>>>
>>>>>      Webrev :
>>>>> https://github.com/srikalyc/JDKfixes/blob/master/java.awt.static_raw_webrev.zip 
>>>>>
>>>>>
>>>>>
>>>>
>>>> For convenient reviewing, I put it to cr.openjdk.java.net:
>>>>
>>>> http://cr.openjdk.java.net/~art/srikalyc/8022184/
>>>>
>>>> Thanks,
>>>>
>>>> Artem
>>>>
>>>>>      Fix       :  static and raw type warnings fix for java.awt 
>>>>> classes
>>>>>
>>



More information about the awt-dev mailing list