Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8252936

Optimize removal of listeners from ExpressionHelper.Generic

    XMLWordPrintable

Details

    • Enhancement
    • Resolution: Unresolved
    • P3
    • tbd
    • jfx11, 8, 9, jfx15
    • javafx

    Description

      ExpressionHelper.Generic is a utility class that is used in the implementation of bindings. When bindings are created and disposed listeners are added and removed from ExpressionHelper.Generic. This is done using an array list, leading to poor performance of List.remove when the list grows too large. Using a set would improve performance. The solution would need to ensure that the performance and memory doesn't suffer for a small number of listeners.

      Attachments

        Issue Links

          duplicates

          Bug - A problem which impairs or prevents the functions of the product. JDK-8185886 Improve scrolling performance of TableView and TreeTableView

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-8252811 The list of cells in a VirtualFlow is cleared every time the number of items changes

          • P3 - Major loss of function.
          • Resolved

          Enhancement - null JDK-8252935 Add treeShowing listener only when needed

          • P3 - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JDK-8185887 TableRowSkinBase fails to correctly virtualize cells in horizontal direction

          • P3 - Major loss of function.
          • Open
          links to

          Review Review openjdk/jfx/108

          Activity

            People

              kcr Kevin Rushforth
              kcr Kevin Rushforth
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: