From d1c72901da8e180fc10ef8279b5da9c19c9374e8 Mon Sep 17 00:00:00 2001 From: Michael Vorburger Date: Fri, 27 Jan 2017 18:32:32 +0100 Subject: [PATCH] mdsalutil-api clean up Checkstyle violations (not enforced yet) org.opendaylight.genius.mdsalutil.matches package is CS clean now. Change-Id: Iea28d160dc90696a676a529630d51b7da95d6091 Signed-off-by: Michael Vorburger --- .../genius/mdsalutil/matches/MatchArpOp.java | 16 +++++++++---- .../genius/mdsalutil/matches/MatchArpSha.java | 18 +++++++++----- .../genius/mdsalutil/matches/MatchArpSpa.java | 16 +++++++++---- .../genius/mdsalutil/matches/MatchArpTha.java | 18 +++++++++----- .../genius/mdsalutil/matches/MatchArpTpa.java | 16 +++++++++---- .../matches/MatchEthernetDestination.java | 20 +++++++++++----- .../matches/MatchEthernetSource.java | 20 +++++++++++----- .../mdsalutil/matches/MatchEthernetType.java | 18 +++++++++----- .../genius/mdsalutil/matches/MatchIcmpv4.java | 24 ++++++++++++------- .../genius/mdsalutil/matches/MatchIcmpv6.java | 24 ++++++++++++------- .../genius/mdsalutil/matches/MatchInPort.java | 22 +++++++++++------ .../mdsalutil/matches/MatchIpProtocol.java | 18 +++++++++----- .../matches/MatchIpv4Destination.java | 16 +++++++++---- .../mdsalutil/matches/MatchIpv4Source.java | 16 +++++++++---- .../matches/MatchIpv6Destination.java | 16 +++++++++---- .../mdsalutil/matches/MatchIpv6NdTarget.java | 18 +++++++++----- .../mdsalutil/matches/MatchIpv6Source.java | 16 +++++++++---- .../mdsalutil/matches/MatchMetadata.java | 20 +++++++++++----- .../mdsalutil/matches/MatchMplsLabel.java | 20 ++++++++++------ .../mdsalutil/matches/MatchPbbIsid.java | 20 ++++++++++------ .../matches/MatchTcpDestinationPort.java | 18 +++++++++----- .../mdsalutil/matches/MatchTcpFlags.java | 16 +++++++++---- .../mdsalutil/matches/MatchTcpSourcePort.java | 18 +++++++++----- .../mdsalutil/matches/MatchTunnelId.java | 20 +++++++++++----- .../matches/MatchUdpDestinationPort.java | 18 +++++++++----- .../mdsalutil/matches/MatchUdpSourcePort.java | 18 +++++++++----- .../mdsalutil/matches/MatchVlanVid.java | 16 +++++++++---- 27 files changed, 336 insertions(+), 160 deletions(-) diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpOp.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpOp.java index 172434b7e..737ba3d60 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpOp.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpOp.java @@ -40,12 +40,18 @@ public class MatchArpOp extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSha.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSha.java index c5c3f21e2..af83e876a 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSha.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSha.java @@ -38,12 +38,18 @@ public class MatchArpSha extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSpa.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSpa.java index 5011a4885..c3d4130e9 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSpa.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpSpa.java @@ -46,12 +46,18 @@ public class MatchArpSpa extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTha.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTha.java index 82088723c..e9dfe9e34 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTha.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTha.java @@ -38,12 +38,18 @@ public class MatchArpTha extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTpa.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTpa.java index d98edd202..5b7a026ff 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTpa.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchArpTpa.java @@ -46,12 +46,18 @@ public class MatchArpTpa extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetDestination.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetDestination.java index 2d208b836..a3efe8bfa 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetDestination.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchEthernetDestination.java @@ -52,14 +52,22 @@ public class MatchEthernetDestination extends MatchInfoHelper>> 32)); + result = 31 * result + (int) (type ^ type >>> 32); return result; } } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIcmpv4.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIcmpv4.java index 86b0ffc62..101a6fbbd 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIcmpv4.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIcmpv4.java @@ -42,22 +42,30 @@ public class MatchIcmpv4 extends MatchInfoHelper>> 32)); + result = 31 * result + (int) (portNumber ^ portNumber >>> 32); return result; } } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpProtocol.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpProtocol.java index 2ada1b03b..80adf2ed1 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpProtocol.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpProtocol.java @@ -42,12 +42,18 @@ public class MatchIpProtocol extends MatchInfoHelper { } @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 { @Override public int hashCode() { int result = super.hashCode(); - result = 31 * result + (int) protocol; + result = 31 * result + protocol; return result; } } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv4Destination.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv4Destination.java index 789409203..1e2a7fdab 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv4Destination.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchIpv4Destination.java @@ -46,12 +46,18 @@ public class MatchIpv4Destination extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchMplsLabel.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchMplsLabel.java index 1d8cf877f..3057a0e87 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchMplsLabel.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchMplsLabel.java @@ -36,12 +36,18 @@ public class MatchMplsLabel extends MatchInfoHelper>> 32)); + result = 31 * result + (int) (label ^ label >>> 32); return result; } } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchPbbIsid.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchPbbIsid.java index 13c0b539a..a0649caf6 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchPbbIsid.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchPbbIsid.java @@ -37,12 +37,18 @@ public class MatchPbbIsid extends MatchInfoHelper>> 32)); + result = 31 * result + (int) (isid ^ isid >>> 32); return result; } } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpDestinationPort.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpDestinationPort.java index 745cf8f05..2bbb8f363 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpDestinationPort.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchTcpDestinationPort.java @@ -37,12 +37,18 @@ public class MatchTcpDestinationPort extends MatchInfoHelper { } @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; } diff --git a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchUdpDestinationPort.java b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchUdpDestinationPort.java index 357e09ede..db9bb01c1 100644 --- a/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchUdpDestinationPort.java +++ b/mdsalutil/mdsalutil-api/src/main/java/org/opendaylight/genius/mdsalutil/matches/MatchUdpDestinationPort.java @@ -37,12 +37,18 @@ public class MatchUdpDestinationPort extends MatchInfoHelper { } @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; } -- 2.36.6