RFR: 8313651: Add 'final' keyword to public property methods in controls
John Hendrikx
jhendrikx at openjdk.org
Fri Aug 18 22:07:34 UTC 2023
On Thu, 17 Aug 2023 23:07:14 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
> In the Control hierarchy, all property accessor methods must be declared `final`.
>
> Added a test to check for missing `final` keyword and added the said keyword where required.
Marked as reviewed by jhendrikx (Committer).
modules/javafx.controls/src/test/java/test/javafx/scene/control/ControlPropertiesTest.java line 216:
> 214: sb.append(c);
> 215: }
> 216: }
Suggestion:
sb.append(prefix);
sb.append(Character.toUpperCase(name.charAt(0)));
sb.append(name, 1, name.length);
-------------
PR Review: https://git.openjdk.org/jfx/pull/1213#pullrequestreview-1585264519
PR Review Comment: https://git.openjdk.org/jfx/pull/1213#discussion_r1298893476
More information about the openjfx-dev
mailing list