Would it be better to import statics from SG2D for SurfaceData? For at
least that one file (and some of its subclasses), they are so closely
tied that they should be able to just refer to the same set of static
constants without having to qualify.
I'm thinking that might make the code more readable as well by getting
rid of a sea of "SunGraphics2D." noise...?
...jim
On 12/1/2011 10:10 AM, Mario Torre wrote:
Il giorno 01/dic/2011, alle ore 00:29, Roman Kennke ha scritto:
Btw, on the other mailing lists they are also discussing about the warnings
day, so I guess I'm not so early ;)
Congratulations, it's probably the first time in your life that you're
early ;-) Seriously, maybe this comes from the fact that today started 9
hours earlier in central Europe than on the US west coast??
Cheers, Roman
Ok, here is my latest masterpiece:
http://cr.openjdk.java.net/~neugens/warning-hackaton/webrev.03/
There are a number of warnings I didn't address, I tried to cleanup
all the SunGraphics2D access and some of the raw type ones.
I used SuppressWarning in a couple of cases and had an opportunity
to use the new Diamond syntax in another one.
I didn't clean the raw type warning in SunGraphics2D though, since
I wanted to study this before just suppress the warning, but didn't have
time yet (I'll probably do in the next days).
There are also a couple of unneeded casts that I removed.
I believe the changes are correct and don't introduce any side effects,
and are generally safe, but of course, I need some good review
and testing help since I'm always very conservative when it comes to
OpenJDK changes.
I'll continue in the next days with some other warnings, for example
I've seen a bunch of related ones in other packages (like sun.awt), but
I'll definitely leave them to a future path if no one fixes them before,
since I really just wanted to clean the java2d namespace for now.
Cheers,
Mario
---
pgp key: http://subkeys.pgp.net/ PGP Key ID: 80F240CF
Fingerprint: BA39 9666 94EC 8B73 27FA FC7C 4086 63E3 80F2 40CF
http://www.ladybug-studio.com
IcedRobot: www.icedrobot.org
Proud GNU Classpath developer: http://www.classpath.org/
Read About us at: http://planet.classpath.org
OpenJDK: http://openjdk.java.net/projects/caciocavallo/
Please, support open standards:
http://endsoftpatents.org/
