Fixing compiler warnings in src/demo/share/jfc

[David Holmes]

Hi Marc,

Please take this to the swing-dev at openjdk.java.net mailing list as these 
are not build issues.

Note some/many mailing lists strip attachments.

Thanks,
David

On 24/10/2019 3:59 am, Marc Hoffmann wrote:
> MOTIVATION
> 
> As a developer of the JaCoCo code coverage library I do lots of JDK builds. JDK
> builds are simple, fast and produce minimal log output. Nice! What annoys me
> though are plenty of compiler warnings at the end of the build caused by the
> example code in src/demo/share/jfc
> 
> FIX
> 
> I propose a series of 3 patches (based on each other) which fixes all compiler
> warnings for the demos:
> 
> patch1.txt - Fix compiler warnings in demos: raw types
> patch2.txt - Fix compiler warnings in demos: deprecated APIs
> patch3.txt - Fix compiler warnings in demos: deprecated Applet APIs
> 
> While patch 1 & 2 do not change functionality patch 3 actually removes the
> Applet versions of some of the demos. The java main versions of the same demos
> are still intact.
> 
> The patches are based on changeset 56699:70e6b0d8db13.
> 
> They have been tested from this clone: https://github.com/marchof/jdk/tree/fix-compiler-warnings-in-demos
> 
> RESULT
> 
> All compiler warnings on demo code during JDK build are removed
> 
> TESTING
> 
> I haven't found any automated tests so I manually launched all the demos. From
> what I can say they are still functional.
> 
> SCOPE
> 
> I applied minimal changes to remove compiler warnings only. There are many more
> cleanup opportunities in the demo code. Also there is (dead?) code in
> src/demo/share/java2d which has similar issues. Both are not on scope of these
> patches.
> 
> NEXT STEPS
> 
> I’m have no experience with OpenJDK patches. If you’re interested in getting these warnings fixed please
> let me know how I can submit these patches properly.
> 
> 
> 
>