Clean up version setters
[openflowplugin.git] / openflowjava / openflow-protocol-impl / src / main / java / org / opendaylight / openflowjava / protocol / impl / deserialization / factories / TableModInputMessageFactory.java
index 425bc579e6c0be8440bf9d78f146a1c1adf6e22f..019ff3e85ed3be31eecba1c3ecbf3f602bde7098 100644 (file)
@@ -7,6 +7,9 @@
  */
 package org.opendaylight.openflowjava.protocol.impl.deserialization.factories;
 
+import static org.opendaylight.yangtools.yang.common.netty.ByteBufUtils.readUint32;
+import static org.opendaylight.yangtools.yang.common.netty.ByteBufUtils.readUint8;
+
 import io.netty.buffer.ByteBuf;
 import org.opendaylight.openflowjava.protocol.api.extensibility.OFDeserializer;
 import org.opendaylight.openflowjava.protocol.api.util.EncodeConstants;
@@ -16,8 +19,9 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731
 import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731.TableModInputBuilder;
 
 /**
- * @author giuseppex.petralia@intel.com
+ * Translates TableModInput messages.
  *
+ * @author giuseppex.petralia@intel.com
  */
 public class TableModInputMessageFactory implements OFDeserializer<TableModInput> {
 
@@ -25,10 +29,10 @@ public class TableModInputMessageFactory implements OFDeserializer<TableModInput
 
     @Override
     public TableModInput deserialize(ByteBuf rawMessage) {
-        TableModInputBuilder builder = new TableModInputBuilder();
-        builder.setVersion((short) EncodeConstants.OF13_VERSION_ID);
-        builder.setXid(rawMessage.readUnsignedInt());
-        builder.setTableId(new TableId((long) rawMessage.readUnsignedByte()));
+        TableModInputBuilder builder = new TableModInputBuilder()
+                .setVersion(EncodeConstants.OF_VERSION_1_3)
+                .setXid(readUint32(rawMessage))
+                .setTableId(new TableId(readUint8(rawMessage).toUint32()));
         rawMessage.skipBytes(PADDING_IN_TABLE_MOD_MESSAGE);
         builder.setConfig(createTableConfig(rawMessage.readUnsignedInt()));
         return builder.build();