Unnecessary call to getParent

XMLWordPrintable

    • Type: Enhancement
    • Resolution: Fixed
    • Priority: P4
    • 8
    • Affects Version/s: 8
    • Component/s: javafx

      There's an unneeded call to getParent because the value is already stored in the variable

      diff --git a/modules/graphics/src/main/java/javafx/scene/Node.java b/modules/graphics/src/main/java/javafx/scene/Node.java
      --- a/modules/graphics/src/main/java/javafx/scene/Node.java
      +++ b/modules/graphics/src/main/java/javafx/scene/Node.java
      @@ -7605,7 +7605,7 @@
               boolean isTreeVisible = isVisible();
               if (isTreeVisible) {
                   final Parent p = getParent();
      - isTreeVisible = p != null ? getParent().impl_isTreeVisible() :
      + isTreeVisible = p != null ? p.impl_isTreeVisible() :
                           clipParent != null ? clipParent.impl_isTreeVisible() :
                           getSubScene() == null || getSubScene().impl_isTreeVisible();
               }

            Assignee:
            Pavel Ĺ afrata
            Reporter:
            Tom Schindl
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:
              Imported: