RFR: 8170720: VetoableListDecorator: Indexes to remove are not aggregated

Lukasz Kostyra lkostyra at openjdk.org
Fri Jan 24 15:17:52 UTC 2025


On Tue, 14 Jan 2025 01:56:10 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

> `VetoableListDecorator` does not correctly report removed indices. Instead of reporting ranges like `[0, 3)`, it always reports a pairwise list of single removals like `[0, 1, 1, 2, 2, 3)`.
> 
> The reason is a off-by-one error, and the fix is simple.

LGTM, fix is trivial and tests all work as intended

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

Marked as reviewed by lkostyra (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1675#pullrequestreview-2572930281


More information about the openjfx-dev mailing list