RFR: 8196065: ListChangeListener getRemoved() returns items that were not removed. [v10]

Michael Strauß mstrauss at openjdk.java.net
Sun Jun 27 12:12:19 UTC 2021


On Sun, 27 Jun 2021 10:40:39 GMT, Marius Hanl <mhanl at openjdk.org> wrote:

>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   changes per review
>
> modules/javafx.controls/src/main/java/javafx/scene/control/ControlUtils.java line 109:
> 
>> 107: 
>> 108:             @Override public int getRemovedSize() {
>> 109:                 checkState();
> 
> Is this needed here? Just wondering because it was not there before.

Without the check, it would be possible to inspect the size without calling `next()` first.

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

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


More information about the openjfx-dev mailing list