Ipv4AddressRange: remove unnecessary double->int convertion 36/8336/2
authorMarek Gradzki <[email protected]>
Thu, 7 Sep 2017 11:18:22 +0000 (13:18 +0200)
committerMarek Gradzki <[email protected]>
Thu, 7 Sep 2017 11:23:09 +0000 (11:23 +0000)
Should fix https://sonar.fd.io/issues/search#issues=AViCMnel_ATy1iJVe-79

Change-Id: I798930dea057cac93ab95b6c1d6f660a21d84e74
Signed-off-by: Marek Gradzki <[email protected]>
vpp-common/vpp-translate-utils/src/main/java/io/fd/hc2vpp/common/translate/util/Ipv4AddressRange.java

index 5f014d0..0a8398a 100644 (file)
@@ -20,16 +20,12 @@ import java.util.Objects;
 import javax.annotation.Nonnull;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv4AddressNoZone;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.Ipv4Prefix;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * IPv4 address range representation.
  */
 public final class Ipv4AddressRange {
 
-    private static final Logger LOG = LoggerFactory.getLogger(Ipv4AddressRange.class);
-
     private final Ipv4AddressNoZone start;
     private final Ipv4AddressNoZone end;
 
@@ -100,10 +96,10 @@ public final class Ipv4AddressRange {
 
         // Take care of the rest
         if (prefixLength != 0) {
-            final int mask0 = (byte) (Math.pow(2, prefixLength) - 1) << (8 - prefixLength);
+            final int mask0 = (byte) ((1 << prefixLength) - 1) << (8 - prefixLength);
             prefixAddrBytes0[index] = (byte) (prefixAddrBytes[index] & mask0);
 
-            final int maskF = (byte) (Math.pow(2, 8 - prefixLength) - 1);
+            final int maskF = (byte) ((1 << (8 - prefixLength)) - 1);
             prefixAddrBytesF[index] = (byte) (prefixAddrBytes[index] | maskF);
 
             index++;