Remove unneded check 66/94366/3
authorRobert Varga <robert.varga@pantheon.tech>
Sun, 3 Jan 2021 15:25:26 +0000 (16:25 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Sun, 3 Jan 2021 16:22:05 +0000 (17:22 +0100)
Sonar is pointing out we are using boolean literals, which is not
needed as the check already is a boolean.

Change-Id: Ieeebe144a91d08dd55a6f7971baa03086e6ce13d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
extension/openflowjava-extension-eric/src/main/java/org/opendaylight/openflowjava/eric/codec/match/EricHeader.java

index bde655e71840f8a82aa8c31e56cead96168f0a7f..c75f0388da9686e531dcad44c291e51be9cf1b6f 100644 (file)
@@ -5,7 +5,6 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-
 package org.opendaylight.openflowjava.eric.codec.match;
 
 import com.google.common.primitives.Ints;
@@ -21,15 +20,15 @@ public class EricHeader {
     private final boolean hasMask;
     private final int length;
 
-    public EricHeader(long header) {
+    public EricHeader(final long header) {
         this.headerAsLong = header;
         this.oxmClass = Ints.checkedCast(extractSub(header, 16, 16));
         this.ericField = Ints.checkedCast(extractSub(header, 7, 9));
-        this.hasMask = extractSub(header, 1, 8) == 1 ? true : false;
+        this.hasMask = extractSub(header, 1, 8) == 1;
         this.length = Ints.checkedCast(extractSub(header, 8, 0));
     }
 
-    public EricHeader(int oxmClass, int ericField, boolean hasMask, int length) {
+    public EricHeader(final int oxmClass, final int ericField, final boolean hasMask, final int length) {
         this.oxmClass = oxmClass;
         this.ericField = ericField;
         this.hasMask = hasMask;
@@ -72,7 +71,7 @@ public class EricHeader {
     }
 
     @Override
-    public boolean equals(Object obj) {
+    public boolean equals(final Object obj) {
         if (this == obj) {
             return true;
         }