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

srikalyan chandrashekar srikalyan.chandrashekar at oracle.com
Fri Aug 23 10:24:43 PDT 2013


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.


--
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
>>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20130823/cdee6169/attachment.html 


More information about the awt-dev mailing list