Enforce pcep-spi checkstyle

[bgpcep.git] / pcep / spi / src / main / java / org / opendaylight / protocol / pcep / spi / AbstractEROPathKeySubobjectParser.java

diff --git a/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/AbstractEROPathKeySubobjectParser.java b/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/AbstractEROPathKeySubobjectParser.java
index e93f57ad684846d3e9640e7e3c05da530ef12de9..5bfc1ef9f332ea47264dee45b6604c77ef54e1cd 100644 (file)
--- a/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/AbstractEROPathKeySubobjectParser.java
+++ b/pcep/spi/src/main/java/org/opendaylight/protocol/pcep/spi/AbstractEROPathKeySubobjectParser.java
@@ -5,12 +5,11 @@
  * 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.protocol.pcep.spi;
 
+import static com.google.common.base.Preconditions.checkArgument;
 import static org.opendaylight.protocol.util.ByteBufWriteUtil.writeUnsignedShort;
 
-import com.google.common.base.Preconditions;
 import io.netty.buffer.ByteBuf;
 import io.netty.buffer.Unpooled;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.pcep.types.rev181109.explicit.route.object.ero.Subobject;
@@ -22,7 +21,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.rsvp.rev
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.rsvp.rev150820.explicit.route.subobjects.subobject.type.path.key._case.PathKeyBuilder;
 
 /**
- * Parser for {@link PathKey}
+ * Parser for {@link PathKey}.
  */
 public abstract class AbstractEROPathKeySubobjectParser implements EROSubobjectParser, EROSubobjectSerializer {
 
@@ -33,14 +32,15 @@ public abstract class AbstractEROPathKeySubobjectParser implements EROSubobjectP
     protected static final int CONTENT128_LENGTH = 2 + PCE128_ID_F_LENGTH;
     protected static final int CONTENT_LENGTH = 2 + PCE_ID_F_LENGTH;
 
-    protected abstract byte[] readPceId(final ByteBuf buffer);
+    protected abstract byte[] readPceId(ByteBuf buffer);
 
-    protected abstract void checkContentLenght(final int i) throws PCEPDeserializerException;
+    protected abstract void checkContentLength(int length) throws PCEPDeserializerException;
 
     @Override
     public final Subobject parseSubobject(final ByteBuf buffer, final boolean loose) throws PCEPDeserializerException {
-        Preconditions.checkArgument(buffer != null && buffer.isReadable(), "Array of bytes is mandatory. Can't be null or empty.");
-        checkContentLenght(buffer.readableBytes());
+        checkArgument(buffer != null && buffer.isReadable(),
+                "Array of bytes is mandatory. Can't be null or empty.");
+        checkContentLength(buffer.readableBytes());
         final int pathKey = buffer.readUnsignedShort();
         final byte[] pceId = readPceId(buffer);
         final PathKeyBuilder pBuilder = new PathKeyBuilder();
@@ -54,17 +54,19 @@ public abstract class AbstractEROPathKeySubobjectParser implements EROSubobjectP
 
     @Override
     public final void serializeSubobject(final Subobject subobject, final ByteBuf buffer) {
-        Preconditions.checkArgument(subobject.getSubobjectType() instanceof PathKeyCase, "Unknown subobject instance. Passed %s. Needed PathKey.",
-            subobject.getSubobjectType().getClass());
+        checkArgument(subobject.getSubobjectType() instanceof PathKeyCase,
+            "Unknown subobject instance. Passed %s. Needed PathKey.", subobject.getSubobjectType().getClass());
         final org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.rsvp.rev150820.explicit.route
-            .subobjects.subobject.type.path.key._case.PathKey pk = ((PathKeyCase) subobject.getSubobjectType()).getPathKey();
-        Preconditions.checkArgument(pk.getPceId() != null, "PceId is mandatory.");
-        Preconditions.checkArgument(pk.getPathKey() != null, "PathKey is mandatory.");
+            .subobjects.subobject.type.path.key._case.PathKey pk =
+            ((PathKeyCase) subobject.getSubobjectType()).getPathKey();
+        checkArgument(pk.getPathKey() != null, "PathKey is mandatory.");
         final byte[] pceID = pk.getPceId().getValue();
-        Preconditions.checkArgument(pceID.length == PCE_ID_F_LENGTH || pceID.length == PCE128_ID_F_LENGTH, "PceId 32/128 Bit required.");
+        checkArgument(pceID.length == PCE_ID_F_LENGTH || pceID.length == PCE128_ID_F_LENGTH,
+                "PceId 32/128 Bit required.");
         final ByteBuf body = Unpooled.buffer();
         writeUnsignedShort(pk.getPathKey().getValue(), body);
         body.writeBytes(pceID);
-        EROSubobjectUtil.formatSubobject(pceID.length == PCE_ID_F_LENGTH ? TYPE_32 : TYPE_128, subobject.isLoose(), body, buffer);
+        EROSubobjectUtil.formatSubobject(pceID.length == PCE_ID_F_LENGTH ? TYPE_32 : TYPE_128, subobject.isLoose(),
+                body, buffer);
     }
 }