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