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

Alexey Ivanov aivanov at openjdk.org
Wed Dec 4 20:07:43 UTC 2024


On Wed, 4 Dec 2024 14:51:14 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> `Properties` doesn't allow `null` values.
>> We can replace containsKey+getProperty with getProperty+null check.
>> It's clearer and a bit faster.
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   inline 'key'
>   remove 'final' from 'localized' too

Marked as reviewed by aivanov (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/21824#pullrequestreview-2479792155


More information about the client-libs-dev mailing list