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