Remove unneeded wrapping 64/96964/1
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 20 Jul 2021 18:50:26 +0000 (20:50 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Tue, 20 Jul 2021 18:50:26 +0000 (20:50 +0200)
We are re-wrapping an IpPrefix, let's just reuse it, improving clarity
and performance.

Change-Id: Ib27930c3ab998647a969f45afceea8ff74c93519
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
bgp/topology-provider/src/main/java/org/opendaylight/bgpcep/bgp/topology/provider/LinkstateGraphBuilder.java

index 5e1f1e530b8b94e98c8b7ac7bfcd73f6e61cfc25..21efe88c8a5fe81702aeca04cd660325f441fdbf 100644 (file)
@@ -530,7 +530,7 @@ public class LinkstateGraphBuilder extends AbstractTopologyBuilder<LinkstateRout
         }
 
         /* Create Prefix */
-        PrefixBuilder builder = new PrefixBuilder().setVertexId(vertexId);
+        PrefixBuilder builder = new PrefixBuilder().setVertexId(vertexId).setPrefix(ippfx);
         if (pa.getSrPrefix() != null && pa.getSrPrefix().getSidLabelIndex() instanceof SidCase) {
             builder.setPrefixSid(((SidCase) pa.getSrPrefix().getSidLabelIndex()).getSid());
             if (pa.getSrPrefix().getFlags() instanceof IsisPrefixFlagsCase) {
@@ -544,12 +544,6 @@ public class LinkstateGraphBuilder extends AbstractTopologyBuilder<LinkstateRout
                 builder.setNodeSid(true);
             }
         }
-        if (ippfx.getIpv4Prefix() != null) {
-            builder.setPrefix(new IpPrefix(ippfx.getIpv4Prefix()));
-        }
-        if (ippfx.getIpv6Prefix() != null) {
-            builder.setPrefix(new IpPrefix(ippfx.getIpv6Prefix()));
-        }
         Prefix prefix = builder.build();
 
         /* Add the Prefix to the Connected Vertex within the Connected Graph */