RFR: 8367439: Bulk change notifications for ObservableSet and ObservableMap [v3]
Andy Goryachev
angorya at openjdk.org
Fri Oct 24 18:04:22 UTC 2025
On Fri, 24 Oct 2025 17:23:21 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> since you are already here... saves a pointer maybe
>
> "since you are already here" is not usually sufficient justification for fixing an unrelated bug. So it probably makes sense to do for any new class added, but I'd leave the existing ones alone.
I thought it **is** a new class. But sure, I am ok to keep the code as is.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1885#discussion_r2461459163
More information about the openjfx-dev
mailing list