RFR: 8289384: Fix warnings: method does not override the inherited method since it is private to a different package
Andy Goryachev
duke at openjdk.org
Tue Jul 12 23:04:58 UTC 2022
On Fri, 8 Jul 2022 21:37:47 GMT, Andy Goryachev <duke at openjdk.org> wrote:
> The fix includes:
> - renaming of offending methods to avoid confusion
> - explicitly declaring the offending methods as private
+1 for JLS
While I agree that leaving the code as is does not cause issues (since the methods are not getting overriden), it might fall under maintainability rubric. Especially in the case of BaseShaderGraphics, where a human surely gets confused. Some people do have eclipse and can F3 to see the superclass.
So I would suggest to merge all of my changes.
If this is unacceptable, at least merge the BaseShaderGraphics change, as it is rather confusing.
The good thing is that we only have a very few places, so the probability of confusion due to this issue is rather low.
-------------
PR: https://git.openjdk.org/jfx/pull/824
More information about the openjfx-dev
mailing list