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