Do not shuffle PathArguments needlessly

There is no point in creating a separate collection if we are not
doing anything with it.

Change-Id: I3199c7232c63464b315eff8f69aac98181946ebd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 28b2c230f2277d53dd80892e95122d82276ec871)

diff --git a/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/serializer/AbstractWebsocketSerializer.java b/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/serializer/AbstractWebsocketSerializer.java
index 369e3c97afb0c21eb7ce01745c61116a7d64d2e1..8dfcdef775fc23774d512dec6ec603405673aef5 100644 (file)
--- a/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/serializer/AbstractWebsocketSerializer.java
+++ b/restconf/restconf-common/src/main/java/org/opendaylight/restconf/common/serializer/AbstractWebsocketSerializer.java
@@ -41,14 +41,13 @@ public abstract class AbstractWebsocketSerializer<T extends Exception> {
 
     public final void serialize(final DataTreeCandidate candidate, final boolean leafNodesOnly, final boolean skipData)
             throws T {
-        final Deque<PathArgument> path = new ArrayDeque<>();
-        path.addAll(candidate.getRootPath().getPathArguments());
         if (leafNodesOnly) {
+            final Deque<PathArgument> path = new ArrayDeque<>();
+            path.addAll(candidate.getRootPath().getPathArguments());
             serializeLeafNodesOnly(path, candidate.getRootNode(), skipData);
-            return;
+        } else {
+            serializeData(candidate.getRootPath().getPathArguments(), candidate.getRootNode(), skipData);
         }
-
-        serializeData(path, candidate.getRootNode(), skipData);
     }
 
     void serializeLeafNodesOnly(final Deque<PathArgument> path, final DataTreeCandidateNode candidate,