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