RFR: 8325154: resizeColumnToFitContent is slower than it needs to be
Marius Hanl
mhanl at openjdk.org
Fri Feb 2 08:08:14 UTC 2024
On Fri, 2 Feb 2024 06:55:36 GMT, Robert Lichtenberger <rlichten at openjdk.org> wrote:
> The PR simply moves column and view-updates outside the loop. Since the column or view never changes within the for-loop it is not necessary to call these again and again.
modules/javafx.controls/src/main/java/javafx/scene/control/skin/TableColumnHeader.java line 668:
> 666: tableRow.updateIndex(row);
> 667:
> 668: cell.updateTableRow(tableRow);
can this also be moved out? `cell.updateTableRow(tableRow);`
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1358#discussion_r1475699942
More information about the openjfx-dev
mailing list