core: Undo stateid4.forBytes change

... to reduce the amount of changes in #162

Signed-off-by: Christian Kohlschütter <[email protected]>

Author: kohlschuetter

core/src/main/java/org/dcache/nfs/v4/NFSv4StateHandler.java

@@ -481,7 +481,7 @@ public stateid4 createStateId(NFS4Client client, byte type, int count) {
         // we eat the first 8 bits if the counter, however, we don't expect 16M states be active at the same time,
         // thus the probability of a collision is too low
         Bytes.putInt(other, 8, count << 8 | (type & 0xFF));
-        return stateid4.forBytes(other, STATE_INITIAL_SEQUENCE);
+        return new stateid4(other, STATE_INITIAL_SEQUENCE);
     }
 
     /**

core/src/main/java/org/dcache/nfs/v4/Stateids.java

@@ -56,18 +56,18 @@ private Stateids() {
     }
 
     private final static stateid4 CURRENT_STATEID =
-            stateid4.forBytes(new byte[] {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, 1);
+            new stateid4(new byte[] {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, 1);
     private final static stateid4 INVAL_STATEID =
-            stateid4.forBytes(new byte[] {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, nfs4_prot.NFS4_UINT32_MAX);
+            new stateid4(new byte[] {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, nfs4_prot.NFS4_UINT32_MAX);
     private final static stateid4 ZERO_STATEID =
-            stateid4.forBytes(new byte[] {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, 0);
+            new stateid4(new byte[] {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}, 0);
 
-    private final static stateid4 ONE_STATEID = stateid4.forBytes(new byte[] {
+    private final static stateid4 ONE_STATEID = new stateid4(new byte[] {
             (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff,
             (byte) 0xff, (byte) 0xff, (byte) 0xff, (byte) 0xff}, nfs4_prot.NFS4_UINT32_MAX);
 
     public static stateid4 uptodateOf(stateid4 stateid) {
-        return stateid4.forBytes(stateid.getOpaque().toBytes(), 0);
+        return new stateid4(stateid.getOpaque().toBytes(), 0);
     }
 
     public static stateid4 currentStateId() {

core/src/main/java/org/dcache/nfs/v4/xdr/stateid4.java

@@ -39,16 +39,11 @@ public class stateid4 implements XdrAble, Serializable, Cloneable {
     private byte[] other; // only declared for Java Serialization
     private transient Opaque opaque;
 
-    public static stateid4 forBytes(byte[] bytes, int seqid) {
-        return new stateid4(seqid, Opaque.forBytes(bytes));
-    }
-
     public stateid4(XdrDecodingStream xdr) throws OncRpcException, IOException {
         this.seqid = xdr.xdrDecodeInt();
         this.opaque = Opaque.forBytes(xdr.xdrDecodeOpaque(12));
     }
 
-    @Deprecated(forRemoval = true)
     public stateid4(byte[] bytes, int seqid) {
         this(seqid, Opaque.forBytes(bytes));
     }

core/src/test/java/org/dcache/nfs/v4/xdr/stateid4Test.java

@@ -29,9 +29,9 @@ public class stateid4Test {
     @Test
     public void testEqualsTrue() {
 
-        stateid4 stateidA = stateid4.forBytes("state".getBytes(), 1);
+        stateid4 stateidA = new stateid4("state".getBytes(), 1);
 
-        stateid4 stateidB = stateid4.forBytes("state".getBytes(), 1);
+        stateid4 stateidB = new stateid4("state".getBytes(), 1);
 
         assertTrue("equal keys not equal", stateidA.equals(stateidB));
         assertTrue("equal, but different hashCode", stateidA.hashCode() == stateidB.hashCode());
@@ -41,17 +41,17 @@ public void testEqualsTrue() {
     @Test
     public void testEqualsSame() {
 
-        stateid4 stateidA = stateid4.forBytes("state".getBytes(), 1);
+        stateid4 stateidA = new stateid4("state".getBytes(), 1);
 
         assertTrue("equal keys not equal", stateidA.equals(stateidA));
     }
 
     @Test
     public void testDifferSequence() {
 
-        stateid4 stateidA = stateid4.forBytes("state".getBytes(), 1);
+        stateid4 stateidA = new stateid4("state".getBytes(), 1);
 
-        stateid4 stateidB = stateid4.forBytes("state".getBytes(), 2);
+        stateid4 stateidB = new stateid4("state".getBytes(), 2);
 
         assertTrue("differ by sequence should still be equal", stateidA.equals(stateidB));
         assertFalse("differ by sequence can't be equal", stateidA.equalsWithSeq(stateidB));
@@ -60,9 +60,9 @@ public void testDifferSequence() {
     @Test
     public void testDifferOther() {
 
-        stateid4 stateidA = stateid4.forBytes("stateA".getBytes(), 1);
+        stateid4 stateidA = new stateid4("stateA".getBytes(), 1);
 
-        stateid4 stateidB = stateid4.forBytes("stateB".getBytes(), 1);
+        stateid4 stateidB = new stateid4("stateB".getBytes(), 1);
 
         assertFalse("differ by other not detected", stateidA.equals(stateidB));
     }