On Mon, 15 Nov 2021 13:28:40 GMT, Jeanette Winzenburg <faste...@openjdk.org> wrote:
>> My PR is already merged, so this is not a problem. :) >> I dont know, but since this is only fixing a (also before) wrong comment it >> might be okay as it is very minor? :) > > FYI: now the listener registration - including the incorrect code comment > (which is the same as in current master) - is back at the old location in the > re-inserted setupTreeTableViewListeners. So still need input whether it's > okay to correct the code comment here. I think, it is okay to fix the comment in this review itself. It is very minor to warrant a separate PR. Also, this anomaly was discovered in this review makes it a candidate to be fixed here. ------------- PR: https://git.openjdk.java.net/jfx/pull/632
