Cleanup VersionMessageWrapper

[openflowjava.git] / openflow-protocol-impl / src / main / java / org / opendaylight / openflowjava / protocol / impl / core / VersionMessageWrapper.java

diff --git a/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/VersionMessageWrapper.java b/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/VersionMessageWrapper.java
index a61298de454f145cf1ff2115e97224ffa96dbc7c..cdbe419c92866c5d658e5757686ece3ea238b3c6 100644 (file)
--- a/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/VersionMessageWrapper.java
+++ b/openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/VersionMessageWrapper.java
@@ -8,6 +8,7 @@
 
 package org.opendaylight.openflowjava.protocol.impl.core;
 
+import com.google.common.base.Preconditions;
 import io.netty.buffer.ByteBuf;
 
 /**
@@ -15,18 +16,17 @@ import io.netty.buffer.ByteBuf;
  * @author michal.polkorab
  */
 public class VersionMessageWrapper {
-
-    private short version;
-    private ByteBuf messageBuffer;
+    private final short version;
+    private final ByteBuf messageBuffer;
 
     /**
      * Constructor
      * @param version version decoded in {@link OFVersionDetector}
      * @param messageBuffer message received from {@link OFFrameDecoder}
      */
-    public VersionMessageWrapper(short version, ByteBuf messageBuffer) {
+    public VersionMessageWrapper(final short version, final ByteBuf messageBuffer) {
         this.version = version;
-        this.messageBuffer = messageBuffer;
+        this.messageBuffer = Preconditions.checkNotNull(messageBuffer);
     }
 
     /**
@@ -42,6 +42,4 @@ public class VersionMessageWrapper {
     public ByteBuf getMessageBuffer() {
         return messageBuffer;
     }
-
-
 }