On Sun, 12 Apr 2026 13:26:21 GMT, Charlie Schlosser <[email protected]> wrote:
>> Are you planning to add more tests? > > @andy-goryachev-oracle @Maran23 I added a test that ensures `getTreeItem` > performs the expected pre-order, depth-first traversal in the original > `TreeUtil.getItem` before and after some of the nodes are collapsed. Created https://bugs.openjdk.org/browse/JDK-8382256 for the cleanup. @chuckyschluz Is this something you want to do? Otherwise I will probably have a look this week. It seems to be only that one method. While here, we could probably add the missing generic in `TreeUtil.updateExpandedItemCount` to remove the warning (there is no reason to not add it there). Also I had a look at `TreeUtil.getRow`, and it looks very unoptimized as well. This might be a good candidate for optimization as well. But has nothing to do with the cleanup. ------------- PR Comment: https://git.openjdk.org/jfx/pull/2142#issuecomment-4252636735
