Use a global comparator

Rather than instantiating a new comparator every time, just reuse a
single instance.

Change-Id: Ida1e9c1f41c5ab67a2847173bdcad6a1cd780c0e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

diff --git a/dom/mdsal-dom-broker/src/main/java/org/opendaylight/mdsal/dom/broker/DOMActionRoutingTableEntry.java b/dom/mdsal-dom-broker/src/main/java/org/opendaylight/mdsal/dom/broker/DOMActionRoutingTableEntry.java
index f44d884b66fc912d7aa000396784a226fa9be9cd..21c872af47cdcd24f9b1e9c88ae1d802d6788489 100644 (file)
--- a/dom/mdsal-dom-broker/src/main/java/org/opendaylight/mdsal/dom/broker/DOMActionRoutingTableEntry.java
+++ b/dom/mdsal-dom-broker/src/main/java/org/opendaylight/mdsal/dom/broker/DOMActionRoutingTableEntry.java
@@ -24,6 +24,9 @@ import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 @Beta
 final class DOMActionRoutingTableEntry extends AbstractDOMRoutingTableEntry<DOMDataTreeIdentifier,
         DOMActionImplementation, AvailabilityListener> {
+    private static final Comparator<DOMActionImplementation> COMPARATOR =
+        Comparator.comparingLong(DOMActionImplementation::invocationCost);
+
     DOMActionRoutingTableEntry(final SchemaPath type,
             final Map<DOMDataTreeIdentifier, List<DOMActionImplementation>> implementations) {
         super(type, implementations);
@@ -37,7 +40,7 @@ final class DOMActionRoutingTableEntry extends AbstractDOMRoutingTableEntry<DOMD
 
     @Override
     protected Comparator<DOMActionImplementation> implComparator() {
-        return Comparator.comparingLong(DOMActionImplementation::invocationCost);
+        return COMPARATOR;
     }
 
     @Override