On Fri, 2 Feb 2024 06:55:36 GMT, Robert Lichtenberger <rlich...@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

Reply via email to