Review Request for 7116993 (warning cleanup java.applet)

Danesh Dadachanji ddadacha at redhat.com
Thu Dec 1 13:39:35 PST 2011


Hi,

On 01/12/11 04:14 PM, Brandon Passanisi wrote:
> I'll withdraw this review request as I was notified by Xiomara that
> someone else has been working on java.applet. I had sent an e-mail to
> the alias yesterday about taking on java.applet though. Luckily, there
> aren't many warnings in java.applet.

Once again, I apologize for the confusion. I assumed sending a private 
email would be fine with the info being on the wiki.

In any case, the changes I made are identical to yours so that's 
promising! One thing that I noticed is that resize(int, int) calls a 
deprecated Component.size() and Component.resize(int, int). Changing the 
calls of size and super.resize to getSize and super.setSize respectively 
might be better. However, that changes functionality. Comments on this?

As Brandon requested, I'll reuse the bug ID (7116993). My webrev is here:

http://icedtea.classpath.org/~ddadacha/warnings-cleanup-java-applet/

Regards,

Danesh


More information about the jdk8-dev mailing list