RFR: 8253086: Optimization of removeAll and retainAll of ObservableListWrapper [v4]

Nir Lisker nlisker at openjdk.java.net
Fri Oct 2 17:18:39 UTC 2020


On Fri, 2 Oct 2020 16:01:20 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> the reason BitSet was introduced was to ensure that the elements are removed from this List in reverse order (prior to
> that fix, they were removed in forward order with the loop index being messed up).

But why do they need to be removed in reverse order to begin with? The super implementation does forward removal, or
just use `removeIf`.

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

PR: https://git.openjdk.java.net/jfx/pull/305


More information about the openjfx-dev mailing list