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