On Mon, 13 Apr 2026 00:05:43 GMT, Charlie Schlosser <[email protected]> wrote:
> The nonsense invalid index is guarded at the top by `if (row < 0) return > null;` Yes, but I meant the situation where we have `index >= expandedItemCount`. This might be worth to check before creating the iterator. But I would prefer that in a follow up, since we need to make sure that we do not break anything. Also we could now cleanup `TreeUtil.getItem` and related methods as they are unused. Again, I would like to see that in a follow up PR, since it looks like we can clean a bit more then. I can create a ticket when this PR is merged! ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2142#discussion_r3072870420
