Do not use 'new Long()' 24/70624/5
authorRobert Varga <robert.varga@pantheon.tech>
Sat, 7 Apr 2018 11:57:22 +0000 (13:57 +0200)
committerClaudio David Gasparini <claudio.gasparini@pantheon.tech>
Thu, 26 Apr 2018 09:14:56 +0000 (09:14 +0000)
Instantiating a long through new is inefficient, use Long.valueOf()
instead.

Change-Id: I7c520334e3a0012e7938a11fc587bfa894afbcf4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
bgp/linkstate/src/main/java/org/opendaylight/protocol/bgp/linkstate/impl/attribute/sr/SidLabelIndexParser.java

index 64f41d31bd85c1097eae9449a87004d2fe135687..21d9de00fa7a5f213e1830b5274f4149fd817fc8 100644 (file)
@@ -82,7 +82,7 @@ public final class SidLabelIndexParser {
     public static SidLabelIndex parseSidLabelIndex(final Size length, final ByteBuf buffer) {
         switch (length) {
         case LABEL:
-            return new LocalLabelCaseBuilder().setLocalLabel(new MplsLabel(new Long(buffer.readUnsignedMedium() & LABEL_MASK))).build();
+            return new LocalLabelCaseBuilder().setLocalLabel(new MplsLabel(Long.valueOf(buffer.readUnsignedMedium() & LABEL_MASK))).build();
         case SID:
             return new SidCaseBuilder().setSid(buffer.readUnsignedInt()).build();
         case IPV6_ADD: