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
