[OpenJDK 2D-Dev] OpenJDK Warning hackathon :) - patch 1
philip.race at oracle.com
Wed Nov 30 20:04:35 UTC 2011
Is this a default javac warning or just from the lint option ?
Any way, fair enough, fix looks fine, although you seem to be getting
an early start on warnings day.
I think its right to handle these on the 2D list as that's the right
place to ask for a review -- and to push into the 2D forest.
I suggest that if you have a bunch of these that you
make minimise the number of patches, hence the number of commits .. and
the number of bug IDs. Semi-related or similar fixes under one ID, resort to
a new one for less related changes.
BTW the warnings day seems to have decided to focus on java.* classes
but I have no issue with others being fixed. Just declare your intent as to
where you are looking to avoid duplication.
On 11/30/2011 11:43 AM, Mario Torre wrote:
> Il giorno 30/nov/2011, alle ore 20:21, Phil Race ha scritto:
>> What is the warning that this fixes ? What is the reasoning ?
>> I'm supposing referencing a class constant via an instance is
>> somehow considered bad form but who made that call ?
>> On 11/30/2011 10:57 AM, Mario Torre wrote:
>>> This is the patch against the main jdk8 forest repository:
>>> Btw, there is also an unused import which doesn't really issues a
>>> warning but I thought to clean it as well.
> Hi Phil,
> On the jdk8-dev (I should probably have CC them then, but I didn't want to cross post,
> but I indeed assumed context which was in another mailing list, sorry...) Xiomara
> sent a mail about warnings cleanup and I just picked up some of the j2d since I'm more familiar
> with this code.
> The compiler issues warning about those classes:
> [static] static variable should be qualified by type name, SunGraphics2D, instead of an extpression.
> I have more of those, I just thought to send them in separate patches while I go class by class.
> pgp key: http://subkeys.pgp.net/ PGP Key ID: 80F240CF
> Fingerprint: BA39 9666 94EC 8B73 27FA FC7C 4086 63E3 80F2 40CF
> 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:
More information about the 2d-dev