RFR: 8367439: Bulk change notifications for ObservableSet and ObservableMap [v3]
Kevin Rushforth
kcr at openjdk.org
Fri Oct 24 17:26:29 UTC 2025
On Fri, 24 Oct 2025 16:20:29 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Yes, I can do that for the existing `Change` implementations, but it's not directly related to bulk change notifications. Maybe another PR would be better.
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1885#discussion_r2461365504
More information about the openjfx-dev
mailing list