RFR: 8345398: Avoid redundant Properties.containsKey call in Cursor.getSystemCustomCursor [v2]

Andrey Turbanov aturbanov at openjdk.org
Wed Dec 4 14:51:14 UTC 2024


On Wed, 4 Dec 2024 13:42:09 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> I really like how this line now fits in 80 characters limit. But if you insist - I will return it.
>
> That's what I thought. At the same time, it's only two columns longer than 80 chars a line.
> 
> The usage of `final` modifiers seems inconsistent in the method. Maybe remove `final` from `localized` too? Then there would be no `final` modifiers till `hotPoint`.

Remove more `final`s. Code _feels_ cleaner without them.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21824#discussion_r1869676986


More information about the client-libs-dev mailing list