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

Move Subscription interface from javafx.beans to javafx.util

XMLWordPrintable

    • Icon: CSR CSR
    • Resolution: Approved
    • Icon: P4 P4
    • jfx21, jfx22
    • javafx
    • None
    • source, binary
    • minimal
    • No risk if integrated together with JDK-8311123 before the final release of JFX 21
    • Java API
    • JDK

      Summary

      The Subscription interface is independent from FX beans and has uses unrelated to beans, it should therefore be placed in javafx.util.

      Problem

      The interface was placed incorrectly. After further consideration it should have been placed in javafx.util.

      Solution

      Move the interface to javafx.util before JDK-8311123 is released.

      Alternatively, remove JDK-8311123 from the current release.

      Specification

      diff --git a/modules/javafx.base/src/main/java/javafx/beans/Subscription.java b/modules/javafx.base/src/main/java/javafx/util/Subscription.java
      similarity index 99%
      rename from modules/javafx.base/src/main/java/javafx/beans/Subscription.java
      rename to modules/javafx.base/src/main/java/javafx/util/Subscription.java
      index d1f96617e9..4bd3dd8af6 100644
      --- a/modules/javafx.base/src/main/java/javafx/beans/Subscription.java
      +++ b/modules/javafx.base/src/main/java/javafx/util/Subscription.java
      @@ -23,7 +23,7 @@
        * questions.
        */
      
      -package javafx.beans;
      +package javafx.util;
      
       import java.util.List;
       import java.util.Objects;

            jhendrikx John Hendrikx
            jhendrikx John Hendrikx
            Kevin Rushforth
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: