[OpenJDK 2D-Dev] JDK 9 RFR of JDK-8033624: Fix raw and unchecked lint warnings in sun.font
Alan Bateman
Alan.Bateman at oracle.com
Thu Feb 6 08:43:58 UTC 2014
On 06/02/2014 07:44, Joe Darcy wrote:
>
> For the range of @SuppressWarnings("unchecked"), if there was already
> a declaration handy, I put the annotation on the declaration (minimal
> scope). If there wasn't a declaration handy and the method was short,
> say less than 10 lines, I put the annotation on the method. Otherwise,
> I introduced a new declaration to host the annotation, roughly
>
> @SuppressWarnings("unchecked")
> Foo tmp = .... // generate the value with the unchecked condition
> existingVariable = tmp; // types match so no warning here
Okay, it looks like you've done as much as possible so thumbs up from me.
-Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20140206/145b85b2/attachment.html>
More information about the 2d-dev
mailing list