JDK 9 RFR of JDK-8032047: Fix static lint warnings in client libraries (original) (raw)

Petr Pchelko petr.pchelko at oracle.com
Fri Jan 17 08:33:03 UTC 2014

Hello, Joe.

The fix looks good.

For the next reviewer - the correct link to the new version is http://cr.openjdk.java.net/~darcy/8032047.1/

With best regards. Petr.

On 17.01.2014, at 11:31, Joe Darcy <joe.darcy at oracle.com> wrote:

Hi Petr,

On 01/16/2014 11:20 PM, Petr Pchelko wrote: Hello, Joe.

A couple of small comments: AffineTransformOp.java: Could you please split a long line #241 Done.

ClipboardTransferable.java: You can safely remove the DataTransferer.getInstance in this place. The DataTransferer is a singleton and it's already created by this time. SunDragSourceContextPeer.java: same here, you can safely remove the getInstance Changed. D3DDrawImage.java: you've mistyped the copyright update. Fixed. D3DSurfaceDataProxy.java: Why did you choose to suppress a warning here? the getD3DDevice is just a getter, it does not allocate new object and has no side effects.. Changed. Revised webrev at http://cr.openjdk.java.net/~darcy/8032047.0/ Thanks for the careful review, -Joe With best regards. Petr. On 17.01.2014, at 10:55, Joe Darcy <joe.darcy at oracle.com> wrote: Hello,

Please view my proposed fix for JDK-8032047: Fix static lint warnings in client libraries http://cr.openjdk.java.net/~darcy/8032047.0/ Mostly this fix consisted of just replacing a variable name with the class name mentioned by javac in the warning message. In a few cases where the expression allocated a new object, I choose to just suppress the warning since I didn't feel motivated to analyze the possible side-effects of the allocation. Thanks, -Joe

More information about the swing-dev mailing list