Integrated: 8289394: Fix warnings: Unlikely argument type

[Andy Goryachev]

On Fri, 8 Jul 2022 20:31:31 GMT, Andy Goryachev <duke at openjdk.org> wrote:

> Fixes "Unlikely argument type" warning generated by the latest Eclipse IDE.
> 
> This warning should be reclassified as an error, as it catches bugs missed by javac.  In this case, the following places seem to contain bugs:
> - apps/samples/3DViewer/src/main/java/com/javafx/experiments/jfx3dviewer/TimelineController.java
> - modules/javafx.graphics/src/main/java/com/sun/glass/ui/monocle/GetEvent.java
> 
> The fixes include:
> - using objects of the right type
> - adding @SuppressWarnings("unlikely-arg-type") in places where intended (e.g. assertFalse)
> 
> There was an earlier discussion about using IDE-specific @SuppressWarnings in the code.  While I might disagree (I think it is ok to use IDE-specific @SuppressWarnings), the tests can be reworked to avoid @SuppressWarnings.  Please let me know.  Thanks!

This pull request has now been integrated.

Changeset: cb7460df
Author:    Andy Goryachev <andy.goryachev at oracle.com>
Committer: Kevin Rushforth <kcr at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/cb7460dfd5e537c457451991e4459eb1044e0956
Stats:     49 lines in 15 files changed: 19 ins; 12 del; 18 mod

8289394: Fix warnings: Unlikely argument type

Reviewed-by: nlisker, kcr

-------------

PR: https://git.openjdk.org/jfx/pull/823