mdsalutil-api clean up Checkstyle violations (not enforced yet) 41/51141/4
authorMichael Vorburger <vorburger@redhat.com>
Fri, 27 Jan 2017 17:32:32 +0000 (18:32 +0100)
committerDavid Suarez <david.suarez.fuentes@ericsson.com>
Mon, 30 Jan 2017 16:06:52 +0000 (16:06 +0000)
org.opendaylight.genius.mdsalutil.matches package is CS clean now.

Change-Id: Iea28d160dc90696a676a529630d51b7da95d6091
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
27 files changed:
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpOp.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSha.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSpa.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTha.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTpa.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetDestination.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetSource.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetType.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIcmpv4.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIcmpv6.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchInPort.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpProtocol.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv4Destination.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv4Source.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv6Destination.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv6NdTarget.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv6Source.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchMetadata.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchMplsLabel.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchPbbIsid.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpDestinationPort.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpFlags.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpSourcePort.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTunnelId.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchUdpDestinationPort.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchUdpSourcePort.java
mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchVlanVid.java

index 172434b7e41154b5625d4228097c402659500529..737ba3d607a2517d62e114c371d9d3f9892a89c5 100644 (file)
@@ -40,12 +40,18 @@ public class MatchArpOp extends MatchInfoHelper<ArpMatch, ArpMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchArpOp that = (MatchArpOp) o;
+        MatchArpOp that = (MatchArpOp) other;
 
         return op == that.op;
     }
index c5c3f21e2bf6a7f7fe12e6132169d80c799b0c66..af83e876a26ad1b6a74bf966697992d1c7c151ff 100644 (file)
@@ -38,12 +38,18 @@ public class MatchArpSha extends MatchInfoHelper<ArpMatch, ArpMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchArpSha that = (MatchArpSha) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchArpSha that = (MatchArpSha) other;
 
         return address != null ? address.equals(that.address) : that.address == null;
     }
index 5011a48853dc5bcf727244222c2a3112ee7a8e4c..c3d4130e9023bad78f07ddcf6cd606b328eb6186 100644 (file)
@@ -46,12 +46,18 @@ public class MatchArpSpa extends MatchInfoHelper<ArpMatch, ArpMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchArpSpa that = (MatchArpSpa) o;
+        MatchArpSpa that = (MatchArpSpa) other;
 
         return address != null ? address.equals(that.address) : that.address == null;
     }
index 82088723c4585f529a6cd2425d931db2c7bc49a3..e9dfe9e349e27ee4eb3956dac39247e7596785a9 100644 (file)
@@ -38,12 +38,18 @@ public class MatchArpTha extends MatchInfoHelper<ArpMatch, ArpMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchArpTha that = (MatchArpTha) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchArpTha that = (MatchArpTha) other;
 
         return address != null ? address.equals(that.address) : that.address == null;
     }
index d98edd2029a19b45eb3d555b942248a38864fc05..5b7a026ffa8c0d1a1a815e7d7f9345e5c3031fcd 100644 (file)
@@ -46,12 +46,18 @@ public class MatchArpTpa extends MatchInfoHelper<ArpMatch, ArpMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchArpTpa that = (MatchArpTpa) o;
+        MatchArpTpa that = (MatchArpTpa) other;
 
         return address != null ? address.equals(that.address) : that.address == null;
     }
index 2d208b83697ba5a24556128530b30e31d051f19c..a3efe8bfac80f5d2ab3ebef25d07774a9b337d33 100644 (file)
@@ -52,14 +52,22 @@ public class MatchEthernetDestination extends MatchInfoHelper<EthernetMatch, Eth
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchEthernetDestination that = (MatchEthernetDestination) o;
+        MatchEthernetDestination that = (MatchEthernetDestination) other;
 
-        if (address != null ? !address.equals(that.address) : that.address != null) return false;
+        if (address != null ? !address.equals(that.address) : that.address != null) {
+            return false;
+        }
         return mask != null ? mask.equals(that.mask) : that.mask == null;
     }
 
index f53ffccde3cdd263f7056ded11e5bdb6b11359b0..68a93a2b8a4b8d86e3b17610f9807be5efab6b4f 100644 (file)
@@ -52,14 +52,22 @@ public class MatchEthernetSource extends MatchInfoHelper<EthernetMatch, Ethernet
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchEthernetSource that = (MatchEthernetSource) o;
+        MatchEthernetSource that = (MatchEthernetSource) other;
 
-        if (address != null ? !address.equals(that.address) : that.address != null) return false;
+        if (address != null ? !address.equals(that.address) : that.address != null) {
+            return false;
+        }
         return mask != null ? mask.equals(that.mask) : that.mask == null;
     }
 
index 2d7225183351f1367efb3bd3ba31451212cc03d1..8b0851761cb0bec9c06430432aae0fe6d054aa1d 100644 (file)
@@ -45,12 +45,18 @@ public class MatchEthernetType extends MatchInfoHelper<EthernetMatch, EthernetMa
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchEthernetType that = (MatchEthernetType) o;
+        MatchEthernetType that = (MatchEthernetType) other;
 
         return type == that.type;
     }
@@ -58,7 +64,7 @@ public class MatchEthernetType extends MatchInfoHelper<EthernetMatch, EthernetMa
     @Override
     public int hashCode() {
         int result = super.hashCode();
-        result = 31 * result + (int) (type ^ (type >>> 32));
+        result = 31 * result + (int) (type ^ type >>> 32);
         return result;
     }
 }
index 86b0ffc62cd5e3047551070f24a69b4358daedf1..101a6fbbdeb12910a1ff4fad5a32056f9be526d8 100644 (file)
@@ -42,22 +42,30 @@ public class MatchIcmpv4 extends MatchInfoHelper<Icmpv4Match, Icmpv4MatchBuilder
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIcmpv4 that = (MatchIcmpv4) o;
+        MatchIcmpv4 that = (MatchIcmpv4) other;
 
-        if (type != that.type) return false;
+        if (type != that.type) {
+            return false;
+        }
         return code == that.code;
     }
 
     @Override
     public int hashCode() {
         int result = super.hashCode();
-        result = 31 * result + (int) type;
-        result = 31 * result + (int) code;
+        result = 31 * result + type;
+        result = 31 * result + code;
         return result;
     }
 }
index 9a7e8e04393ab904229771537c40e35f6b8dd056..687c9d97a65c8bd3ad23c0464b2b68c835127b17 100644 (file)
@@ -42,22 +42,30 @@ public class MatchIcmpv6 extends MatchInfoHelper<Icmpv6Match, Icmpv6MatchBuilder
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIcmpv6 that = (MatchIcmpv6) o;
+        MatchIcmpv6 that = (MatchIcmpv6) other;
 
-        if (type != that.type) return false;
+        if (type != that.type) {
+            return false;
+        }
         return code == that.code;
     }
 
     @Override
     public int hashCode() {
         int result = super.hashCode();
-        result = 31 * result + (int) type;
-        result = 31 * result + (int) code;
+        result = 31 * result + type;
+        result = 31 * result + code;
         return result;
     }
 }
index bf22e8d6db25051521872ee6141ab9ea0ec5d384..5c130e977b19e32221ffa4ba885196573b7f7259 100644 (file)
@@ -46,14 +46,22 @@ public class MatchInPort extends MatchInfo {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchInPort that = (MatchInPort) o;
+        MatchInPort that = (MatchInPort) other;
 
-        if (portNumber != that.portNumber) return false;
+        if (portNumber != that.portNumber) {
+            return false;
+        }
         return dpId != null ? dpId.equals(that.dpId) : that.dpId == null;
     }
 
@@ -61,7 +69,7 @@ public class MatchInPort extends MatchInfo {
     public int hashCode() {
         int result = super.hashCode();
         result = 31 * result + (dpId != null ? dpId.hashCode() : 0);
-        result = 31 * result + (int) (portNumber ^ (portNumber >>> 32));
+        result = 31 * result + (int) (portNumber ^ portNumber >>> 32);
         return result;
     }
 }
index 2ada1b03b1b9760ce2edca37971563b16b40883b..80adf2ed105e847a00f5cf75fc38a438d00c8835 100644 (file)
@@ -42,12 +42,18 @@ public class MatchIpProtocol extends MatchInfoHelper<IpMatch, IpMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIpProtocol that = (MatchIpProtocol) o;
+        MatchIpProtocol that = (MatchIpProtocol) other;
 
         return protocol == that.protocol;
     }
@@ -55,7 +61,7 @@ public class MatchIpProtocol extends MatchInfoHelper<IpMatch, IpMatchBuilder> {
     @Override
     public int hashCode() {
         int result = super.hashCode();
-        result = 31 * result + (int) protocol;
+        result = 31 * result + protocol;
         return result;
     }
 }
index 789409203d879cd6e683730fa05ba7ac47eb488e..1e2a7fdab7d120ba79e7ba76743cb789f2b3a1b7 100644 (file)
@@ -46,12 +46,18 @@ public class MatchIpv4Destination extends MatchInfoHelper<Ipv4Match, Ipv4MatchBu
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIpv4Destination that = (MatchIpv4Destination) o;
+        MatchIpv4Destination that = (MatchIpv4Destination) other;
 
         return prefix != null ? prefix.equals(that.prefix) : that.prefix == null;
     }
index 0c884c1d620b1ea4f7546fb0022a8dfbddb51dc2..ab411a89c2aee8a81e0882f7e93257ab2118a7f8 100644 (file)
@@ -46,12 +46,18 @@ public class MatchIpv4Source extends MatchInfoHelper<Ipv4Match, Ipv4MatchBuilder
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIpv4Source that = (MatchIpv4Source) o;
+        MatchIpv4Source that = (MatchIpv4Source) other;
 
         return prefix != null ? prefix.equals(that.prefix) : that.prefix == null;
     }
index 2977177a5b7587d9f37fe7ca00965b37b80f25ca..cc7093ce0f762ca14bb4ddb5de95f6a4d1c7dcb7 100644 (file)
@@ -41,12 +41,18 @@ public class MatchIpv6Destination extends MatchInfoHelper<Ipv6Match, Ipv6MatchBu
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIpv6Destination that = (MatchIpv6Destination) o;
+        MatchIpv6Destination that = (MatchIpv6Destination) other;
 
         return prefix != null ? prefix.equals(that.prefix) : that.prefix == null;
     }
index ccaa88b5f87cc8dad170fe016b45297ba9f87092..ac435211946d722205622fb7a5c01b97ba8c00c0 100644 (file)
@@ -37,12 +37,18 @@ public class MatchIpv6NdTarget extends MatchInfoHelper<Ipv6Match, Ipv6MatchBuild
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchIpv6NdTarget that = (MatchIpv6NdTarget) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchIpv6NdTarget that = (MatchIpv6NdTarget) other;
 
         return address != null ? address.equals(that.address) : that.address == null;
     }
index 47bd918d3dca44b13e25230d63066820e00ee2e5..2a12960e30bd956f782ea15d9f1fea3cfd0287f5 100644 (file)
@@ -41,12 +41,18 @@ public class MatchIpv6Source extends MatchInfoHelper<Ipv6Match, Ipv6MatchBuilder
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchIpv6Source that = (MatchIpv6Source) o;
+        MatchIpv6Source that = (MatchIpv6Source) other;
 
         return prefix != null ? prefix.equals(that.prefix) : that.prefix == null;
     }
index 9bccb6ba9da8a0b52118a09105e5ab1afd8b7ec1..d186e293a6a027b6f04bb8c920eef382bc23561b 100644 (file)
@@ -43,14 +43,22 @@ public class MatchMetadata extends MatchInfoHelper<Metadata, MetadataBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchMetadata that = (MatchMetadata) o;
+        MatchMetadata that = (MatchMetadata) other;
 
-        if (metadata != null ? !metadata.equals(that.metadata) : that.metadata != null) return false;
+        if (metadata != null ? !metadata.equals(that.metadata) : that.metadata != null) {
+            return false;
+        }
         return mask != null ? mask.equals(that.mask) : that.mask == null;
     }
 
index 1d8cf877f52ae3f06deccbbaa4457a1834bd3984..3057a0e872e90fdc29527872ec9579e1009180c4 100644 (file)
@@ -36,12 +36,18 @@ public class MatchMplsLabel extends MatchInfoHelper<ProtocolMatchFields, Protoco
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchMplsLabel that = (MatchMplsLabel) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchMplsLabel that = (MatchMplsLabel) other;
 
         return label == that.label;
     }
@@ -49,7 +55,7 @@ public class MatchMplsLabel extends MatchInfoHelper<ProtocolMatchFields, Protoco
     @Override
     public int hashCode() {
         int result = super.hashCode();
-        result = 31 * result + (int) (label ^ (label >>> 32));
+        result = 31 * result + (int) (label ^ label >>> 32);
         return result;
     }
 }
index 13c0b539a23205e14b5ff4973e432a570119c2bf..a0649caf68529f64c6e22a1199b5799d5e13aee0 100644 (file)
@@ -37,12 +37,18 @@ public class MatchPbbIsid extends MatchInfoHelper<ProtocolMatchFields, ProtocolM
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchPbbIsid that = (MatchPbbIsid) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchPbbIsid that = (MatchPbbIsid) other;
 
         return isid == that.isid;
     }
@@ -50,7 +56,7 @@ public class MatchPbbIsid extends MatchInfoHelper<ProtocolMatchFields, ProtocolM
     @Override
     public int hashCode() {
         int result = super.hashCode();
-        result = 31 * result + (int) (isid ^ (isid >>> 32));
+        result = 31 * result + (int) (isid ^ isid >>> 32);
         return result;
     }
 }
index 745cf8f05d93c5937ca9f653357910db8c4f4073..2bbb8f363c9d338b8f29bbe354a8652eb0575293 100644 (file)
@@ -37,12 +37,18 @@ public class MatchTcpDestinationPort extends MatchInfoHelper<TcpMatch, TcpMatchB
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchTcpDestinationPort that = (MatchTcpDestinationPort) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchTcpDestinationPort that = (MatchTcpDestinationPort) other;
 
         return port == that.port;
     }
index 80ecda2001299dfd9bb880c72d5c4deff8b352ef..ec62a5247fc963a08a573649da57b0c68881f901 100644 (file)
@@ -40,12 +40,18 @@ public class MatchTcpFlags extends MatchInfoHelper<TcpFlagsMatch, TcpFlagsMatchB
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchTcpFlags that = (MatchTcpFlags) o;
+        MatchTcpFlags that = (MatchTcpFlags) other;
 
         return flags == that.flags;
     }
index 516019592b6b34551dc108b4ee0277e36c5207ec..c628d89e2476daa158d86fecede90188d84899d7 100644 (file)
@@ -37,12 +37,18 @@ public class MatchTcpSourcePort extends MatchInfoHelper<TcpMatch, TcpMatchBuilde
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchTcpSourcePort that = (MatchTcpSourcePort) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchTcpSourcePort that = (MatchTcpSourcePort) other;
 
         return port == that.port;
     }
index dc2ff20df3c0be8e7102cbb20296ed7bd148cc98..17c0c68fef1af24f6bd864e6bfc79fa85df3c308 100644 (file)
@@ -51,14 +51,22 @@ public class MatchTunnelId extends MatchInfoHelper<Tunnel, TunnelBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchTunnelId that = (MatchTunnelId) o;
+        MatchTunnelId that = (MatchTunnelId) other;
 
-        if (tunnelId != null ? !tunnelId.equals(that.tunnelId) : that.tunnelId != null) return false;
+        if (tunnelId != null ? !tunnelId.equals(that.tunnelId) : that.tunnelId != null) {
+            return false;
+        }
         return tunnelMask != null ? tunnelMask.equals(that.tunnelMask) : that.tunnelMask == null;
     }
 
index 357e09ede8e2f2002f316ef0e2df4098f17e3ddb..db9bb01c1868257e7c7b962422aaee635e3b0273 100644 (file)
@@ -37,12 +37,18 @@ public class MatchUdpDestinationPort extends MatchInfoHelper<UdpMatch, UdpMatchB
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchUdpDestinationPort that = (MatchUdpDestinationPort) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchUdpDestinationPort that = (MatchUdpDestinationPort) other;
 
         return port == that.port;
     }
index 95e72c5a07779e951000b329394c2db65b917609..ddf2eeb645361a572034adbd7872e1df9d7aec82 100644 (file)
@@ -37,12 +37,18 @@ public class MatchUdpSourcePort extends MatchInfoHelper<UdpMatch, UdpMatchBuilde
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
-
-        MatchUdpSourcePort that = (MatchUdpSourcePort) o;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
+
+        MatchUdpSourcePort that = (MatchUdpSourcePort) other;
 
         return port == that.port;
     }
index b5105a8a5ec5faca7ae2ab2b1426420954315d12..ff02d742b62c3ef70f76a59fc775173be01c5c6f 100644 (file)
@@ -41,12 +41,18 @@ public class MatchVlanVid extends MatchInfoHelper<VlanMatch, VlanMatchBuilder> {
     }
 
     @Override
-    public boolean equals(Object o) {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        if (!super.equals(o)) return false;
+    public boolean equals(Object other) {
+        if (this == other) {
+            return true;
+        }
+        if (other == null || getClass() != other.getClass()) {
+            return false;
+        }
+        if (!super.equals(other)) {
+            return false;
+        }
 
-        MatchVlanVid that = (MatchVlanVid) o;
+        MatchVlanVid that = (MatchVlanVid) other;
 
         return vlanId == that.vlanId;
     }