On Fri, 3 Dec 2021 11:25:43 GMT, Johan Vos <j...@openjdk.org> wrote:

>> After (re)setting the number of elements, make sure to do at least some 
>> estimation of the total size.
>> Added a testcase for this scenario.
>
> Johan Vos has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   process reviewer comment (simplify call)

The fix and new test looks fine. I added one comment/question regarding the 
existing tests that check a count, but I'm OK with either answer.

Approved.

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

Marked as reviewed by kcr (Lead).

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

Reply via email to