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

Remove use of deprecated finalize methods from javafx property objects

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • P3
    • Resolution: Fixed
    • jfx11, 9, 10
    • jfx15
    • javafx

    Description

      The implementation of the asObject and xxxxxProperty methods in the following classes uses the deprecated Object::finalize method to clean up the property bindings that are created, to avoid a memory leak:

      javafx.beans.property.BooleanProperty
      javafx.beans.property.DoubleProperty
      javafx.beans.property.FloatProperty
      javafx.beans.property.IntegerProperty
      javafx.beans.property.LongProperty

      We should instead use an alternative disposal mechanism, such as a weak or phantom reference-based Disposer used in other places in JavaFX.

      Attachments

        Issue Links

          blocks

          Task - A task that needs to be done. JDK-8229842 ☂ Eliminate use of deprecated finalize method in JavaFX

          • P3 - Major loss of function.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-8196587 Remove use of deprecated finalize method from JPEGImageLoader

          • P3 - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JDK-8235772 Remove use of deprecated finalize method from PiscesRenderer and AbstractSurface

          • P3 - Major loss of function.
          • Resolved

          Enhancement - null JDK-8240692 Cleanup of the javafx property objects

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

          Web Link Commit openjdk/jfx/9cd6f791

          Review Review openjdk/jfx/113

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved: