On Mon, 6 Apr 2026 12:24:23 GMT, Marius Hanl <[email protected]> wrote:
>> Charlie Schlosser has updated the pull request incrementally with one >> additional commit since the last revision: >> >> fix typo in comments > > modules/javafx.controls/src/test/java/test/javafx/scene/control/TableViewTest.java > line 4051: > >> 4049: assertEquals(List.of(0, 3, 6), froms); >> 4050: assertEquals(List.of(2, 5, 8), tos); >> 4051: assertEquals(List.of(0, 1), >> removedLists.get(0).stream().map(TablePosition::getRow).toList()); > > Very minor: Here and below are some unneeded whitespaces I agree: even though it _might_ look nicer when some things are aligned, it's not worth doing it long term. A future refactoring will likely break the alignment. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2131#discussion_r3040112172
