Fix/suppress checkstyle warnings 73/60973/3
authorRobert Varga <robert.varga@pantheon.tech>
Tue, 1 Aug 2017 09:22:18 +0000 (11:22 +0200)
committerRobert Varga <nite@hq.sk>
Tue, 1 Aug 2017 16:05:52 +0000 (16:05 +0000)
Some of the argument names are coming from superclasses, add suppression
of those. Others can be easily renamed.

Change-Id: I75e746287fc2278790091185d967dfcc2d16d94e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
15 files changed:
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/AbstractEntrySet.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/AbstractKeySet.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/CNode.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/EntryUtil.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/INode.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/ImmutableEntrySet.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/ImmutableKeySet.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/ImmutableTrieMap.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/LNodeEntry.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/MutableEntrySet.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/MutableIterator.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/MutableKeySet.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/SNode.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/SerializationProxy.java
third-party/triemap/src/main/java/org/opendaylight/yangtools/triemap/TNode.java

index c915945d77c43f58eeedfa82a33337bb5bd70e39..7089a7e15a5254f88fea68d3cae085a3e87f4ea2 100644 (file)
@@ -40,6 +40,7 @@ abstract class AbstractEntrySet<K, V> extends AbstractSet<Entry<K, V>> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public final boolean contains(final Object o) {
         if (!(o instanceof Entry)) {
             return false;
index 277156a5b5a7fc826994971e48b2e0a07c7d3376..ef4f04cd1472283273e6decb7caff8b01480c392 100644 (file)
@@ -39,16 +39,19 @@ abstract class AbstractKeySet<K> extends AbstractSet<K> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public final boolean add(final K e) {
         throw new UnsupportedOperationException();
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public final boolean addAll(final Collection<? extends K> c) {
         throw new UnsupportedOperationException();
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public final boolean contains(final Object o) {
         return map.containsKey(o);
     }
index f6f95298f0b7b885b1702d4c6cbfbed85ede6e93..12a8fb74ed95152db03f955f2cd94891c4d17aec 100644 (file)
@@ -50,7 +50,7 @@ final class CNode<K, V> extends MainNode<K, V> {
     private static <K, V> MainNode<K,V> dual(final SNode<K, V> x, final int xhc, final SNode<K, V> y, final int yhc,
             final int lev, final Gen gen) {
         if (lev >= HASH_BITS) {
-            return new LNode<>(x.k, x.v, y.k, y.v);
+            return new LNode<>(x.key, x.value, y.key, y.value);
         }
 
         final int xidx = (xhc >>> lev) & 0x1f;
index a73797eff3af55720fcfa46880d7870b6113bded..6fe50d5597551486d404c71186d99fbc0a86c21b 100644 (file)
@@ -30,12 +30,12 @@ final class EntryUtil {
     /**
      * Utility implementing {@link Entry#equals(Object)}.
      */
-    static boolean equal(final Object o, final Object key, final Object value) {
-        if (!(o instanceof Entry)) {
+    static boolean equal(final Object obj, final Object key, final Object value) {
+        if (!(obj instanceof Entry)) {
             return false;
         }
 
-        final Entry<?,?> e = (Entry<?,?>)o;
+        final Entry<?,?> e = (Entry<?,?>)obj;
         return key.equals(e.getKey()) && value.equals(e.getValue());
     }
 
index 23046ed0e835f5148c6003ce2b14708ee4666ff5..3e159173ff962799311cbd2c293b8ce22ecbcc6d 100644 (file)
@@ -161,7 +161,7 @@ final class INode<K, V> extends BasicNode {
                         return false;
                     } else if (cnAtPos instanceof SNode) {
                         final SNode<K, V> sn = (SNode<K, V>) cnAtPos;
-                        if (sn.hc == hc && ct.equal(sn.k, k)) {
+                        if (sn.hc == hc && ct.equal(sn.key, k)) {
                             return GCAS(cn, cn.updatedAt(pos, new SNode<>(k, v, hc), gen), ct);
                         }
 
@@ -253,9 +253,9 @@ final class INode<K, V> extends BasicNode {
                     } else if (cnAtPos instanceof SNode) {
                         final SNode<K, V> sn = (SNode<K, V>) cnAtPos;
                         if (cond == null) {
-                            if (sn.hc == hc && ct.equal(sn.k, k)) {
+                            if (sn.hc == hc && ct.equal(sn.key, k)) {
                                 if (GCAS(cn, cn.updatedAt(pos, new SNode<>(k, v, hc), gen), ct)) {
-                                    return Optional.of(sn.v);
+                                    return Optional.of(sn.value);
                                 }
 
                                 return null;
@@ -263,24 +263,24 @@ final class INode<K, V> extends BasicNode {
 
                             return insertDual(ct, cn, pos, sn, k, v, hc, lev);
                         } else if (cond == ABSENT) {
-                            if (sn.hc == hc && ct.equal(sn.k, k)) {
-                                return Optional.of(sn.v);
+                            if (sn.hc == hc && ct.equal(sn.key, k)) {
+                                return Optional.of(sn.value);
                             }
 
                             return insertDual(ct, cn, pos, sn, k, v, hc, lev);
                         } else if (cond == PRESENT) {
-                            if (sn.hc == hc && ct.equal(sn.k, k)) {
+                            if (sn.hc == hc && ct.equal(sn.key, k)) {
                                 if (GCAS(cn, cn.updatedAt(pos, new SNode<>(k, v, hc), gen), ct)) {
-                                    return Optional.of(sn.v);
+                                    return Optional.of(sn.value);
                                 }
                                 return null;
                             }
 
                             return Optional.empty();
                         } else {
-                            if (sn.hc == hc && ct.equal(sn.k, k) && cond.equals(sn.v)) {
+                            if (sn.hc == hc && ct.equal(sn.key, k) && cond.equals(sn.value)) {
                                 if (GCAS(cn, cn.updatedAt(pos, new SNode<>(k, v, hc), gen), ct)) {
-                                    return Optional.of(sn.v);
+                                    return Optional.of(sn.value);
                                 }
 
                                 return null;
@@ -394,8 +394,8 @@ final class INode<K, V> extends BasicNode {
                 } else if (sub instanceof SNode) {
                     // 2) singleton node
                     final SNode<K, V> sn = (SNode<K, V>) sub;
-                    if (sn.hc == hc && ct.equal(sn.k, k)) {
-                        return sn.v;
+                    if (sn.hc == hc && ct.equal(sn.key, k)) {
+                        return sn.value;
                     }
 
                     return null;
@@ -418,8 +418,8 @@ final class INode<K, V> extends BasicNode {
     private Object cleanReadOnly(final TNode<K, V> tn, final int lev, final INode<K, V> parent,
             final TrieMap<K, V> ct, final K k, final int hc) {
         if (ct.isReadOnly()) {
-            if (tn.hc == hc && ct.equal(tn.k, k)) {
-                return tn.v;
+            if (tn.hc == hc && ct.equal(tn.key, k)) {
+                return tn.value;
             }
 
             return null;
@@ -475,10 +475,10 @@ final class INode<K, V> extends BasicNode {
                 }
             } else if (sub instanceof SNode) {
                 final SNode<K, V> sn = (SNode<K, V>) sub;
-                if (sn.hc == hc && ct.equal(sn.k, k) && (cond == null || cond.equals(sn.v))) {
+                if (sn.hc == hc && ct.equal(sn.key, k) && (cond == null || cond.equals(sn.value))) {
                     final MainNode<K, V> ncn = cn.removedAt(pos, flag, gen).toContracted(lev);
                     if (GCAS(cn, ncn, ct)) {
-                        res = Optional.of(sn.v);
+                        res = Optional.of(sn.value);
                     } else {
                         res = null;
                     }
index 9f031405e3a0bb141db5ead55f8a849733a98572..141f42c245db4ba0b6a187c2e132d8279c6651d3 100644 (file)
@@ -45,16 +45,19 @@ final class ImmutableEntrySet<K, V> extends AbstractEntrySet<K, V> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean remove(final Object o) {
         throw unsupported();
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean removeAll(final Collection<?> c) {
         throw unsupported();
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean retainAll(final Collection<?> c) {
         throw unsupported();
     }
index a25649e7774b0b42de351b0822876ac0284c3cda..8cfbfecff2066e4316cecb8971f6673ac1ea9c44 100644 (file)
@@ -45,16 +45,19 @@ final class ImmutableKeySet<K> extends AbstractKeySet<K> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean remove(final Object o) {
         throw unsupported();
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean retainAll(final Collection<?> c) {
         throw unsupported();
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean removeAll(final Collection<?> c) {
         throw unsupported();
     }
index f3f63be424b89e483d8011007ca389d23ed0c3d2..79664bf63a31f225b6ea387b29bff0c0d4d41442 100644 (file)
@@ -74,6 +74,7 @@ public final class ImmutableTrieMap<K, V> extends TrieMap<K, V> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public void putAll(final Map<? extends K, ? extends V> m) {
         throw unsupported();
     }
index d20ea18dd0d0df84939b024a97e9b3351e0e1015..6a72ccaae4fb10d0ab9c4cd91e077a4b135b91a1 100644 (file)
@@ -58,8 +58,8 @@ abstract class LNodeEntry<K, V> implements Entry<K, V> {
 
     @SuppressFBWarnings(value = "EQ_UNUSUAL",  justification = "Equality handled by utility methods")
     @Override
-    public final boolean equals(final Object o) {
-        return EntryUtil.equal(o, key, value);
+    public final boolean equals(final Object obj) {
+        return EntryUtil.equal(obj, key, value);
     }
 
     @Override
index c9bdca25c63c476fbebaed7ed06582e020f06d7a..04ddb8818931e9a0b694696e7fca30329ea5b6d0 100644 (file)
@@ -32,6 +32,7 @@ final class MutableEntrySet<K, V> extends AbstractEntrySet<K, V> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean add(final Entry<K, V> e) {
         final K k = e.getKey();
         checkArgument(k != null);
@@ -53,6 +54,7 @@ final class MutableEntrySet<K, V> extends AbstractEntrySet<K, V> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean remove(final Object o) {
         if (!(o instanceof Entry)) {
             return false;
index 988be4099a6529e4741d204c46ae9310fec07b7f..20fb567b90e0f282842481a433088821f6b2b516 100644 (file)
@@ -111,8 +111,8 @@ final class MutableIterator<K, V> extends AbstractIterator<K, V> {
 
         @SuppressFBWarnings(value = "EQ_UNUSUAL",  justification = "Equality handled by utility methods")
         @Override
-        public boolean equals(final Object o) {
-            return EntryUtil.equal(o, getKey(), getValue());
+        public boolean equals(final Object obj) {
+            return EntryUtil.equal(obj, getKey(), getValue());
         }
 
         @Override
index f6d5d2c77231fc1f83e58a20ac5b77e51e3655cd..9cb72b079f8374b11636c4feb6186f379a60cc4f 100644 (file)
@@ -56,6 +56,7 @@ final class MutableKeySet<K> extends AbstractKeySet<K> {
     }
 
     @Override
+    @SuppressWarnings("checkstyle:parameterName")
     public boolean remove(final Object o) {
         return map().remove(o) != null;
     }
index ef3e36a02efc8017fd19c5798337ff3b845b8544..d11a592ea54196210a2d0dc9a80434d2a7a8a503 100644 (file)
@@ -18,26 +18,26 @@ package org.opendaylight.yangtools.triemap;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 final class SNode<K, V> extends BasicNode implements EntryNode<K, V> {
-    final K k;
-    final V v;
+    final K key;
+    final V value;
     final int hc;
 
     SNode(final K key, final V value, final int hc) {
-        this.k = key;
-        this.v = value;
+        this.key = key;
+        this.value = value;
         this.hc = hc;
     }
 
     SNode<K, V> copy() {
-        return new SNode<>(k, v, hc);
+        return new SNode<>(key, value, hc);
     }
 
     TNode<K, V> copyTombed() {
-        return new TNode<>(k, v, hc);
+        return new TNode<>(key, value, hc);
     }
 
     SNode<K, V> copyUntombed() {
-        return new SNode<>(k, v, hc);
+        return new SNode<>(key, value, hc);
     }
 
     @Override
@@ -48,27 +48,27 @@ final class SNode<K, V> extends BasicNode implements EntryNode<K, V> {
 
     @Override
     public K getKey() {
-        return k;
+        return key;
     }
 
     @Override
     public V getValue() {
-        return v;
+        return value;
     }
 
     @Override
     public int hashCode() {
-        return EntryUtil.hash(k, v);
+        return EntryUtil.hash(key, value);
     }
 
     @SuppressFBWarnings(value = "EQ_UNUSUAL",  justification = "Equality handled by utility methods")
     @Override
-    public boolean equals(final Object o) {
-        return EntryUtil.equal(o, k, v);
+    public boolean equals(final Object obj) {
+        return EntryUtil.equal(obj, key, value);
     }
 
     @Override
     public String toString() {
-        return EntryUtil.string(k, v);
+        return EntryUtil.string(key, value);
     }
 }
index 700deacc3d2361a21aec132049354c59983295b7..3557fc41252d0431dd04d88521030b1d203d9ac7 100644 (file)
@@ -38,7 +38,7 @@ final class SerializationProxy implements Externalizable {
     private TrieMap<Object, Object> map;
     private boolean readOnly;
 
-    @SuppressWarnings("checkStyle:redundantModifier")
+    @SuppressWarnings("checkstyle:redundantModifier")
     public SerializationProxy() {
         // For Externalizable
     }
index 7a89e218095c35fd2bd510d4a8d267bc2796e690..591d0b0b7996c2831d91055be2e8ce246cf98276 100644 (file)
@@ -18,26 +18,26 @@ package org.opendaylight.yangtools.triemap;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 
 final class TNode<K, V> extends MainNode<K, V> implements EntryNode<K, V> {
-    final K k;
-    final V v;
+    final K key;
+    final V value;
     final int hc;
 
     TNode(final K key, final V value, final int hc) {
-        this.k = key;
-        this.v = value;
+        this.key = key;
+        this.value = value;
         this.hc = hc;
     }
 
     TNode<K, V> copy() {
-        return new TNode<>(k, v, hc);
+        return new TNode<>(key, value, hc);
     }
 
     TNode<K, V> copyTombed() {
-        return new TNode<>(k, v, hc);
+        return new TNode<>(key, value, hc);
     }
 
     SNode<K, V> copyUntombed() {
-        return new SNode<>(k, v, hc);
+        return new SNode<>(key, value, hc);
     }
 
     @Override
@@ -58,27 +58,27 @@ final class TNode<K, V> extends MainNode<K, V> implements EntryNode<K, V> {
 
     @Override
     public K getKey() {
-        return k;
+        return key;
     }
 
     @Override
     public V getValue() {
-        return v;
+        return value;
     }
 
     @Override
     public int hashCode() {
-        return EntryUtil.hash(k, v);
+        return EntryUtil.hash(key, value);
     }
 
     @SuppressFBWarnings(value = "EQ_UNUSUAL",  justification = "Equality handled by utility methods")
     @Override
-    public boolean equals(final Object o) {
-        return EntryUtil.equal(o, k, v);
+    public boolean equals(final Object obj) {
+        return EntryUtil.equal(obj, key, value);
     }
 
     @Override
     public String toString() {
-        return EntryUtil.string(k, v);
+        return EntryUtil.string(key, value);
     }
 }