From db77f058c92ee3d0b531c474093d6a3d1d81cd25 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 26 May 2014 12:17:41 +0000
Subject: [PATCH] Ensured no accidental protocol break can happen by renaming all ByteArrayBuilder.append*() methods and getting rid of all method overloading.

---
 opends/src/server/org/opends/server/replication/protocol/StartMsg.java |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/opends/src/server/org/opends/server/replication/protocol/StartMsg.java b/opends/src/server/org/opends/server/replication/protocol/StartMsg.java
index 9cc0aad..30b6bfa 100644
--- a/opends/src/server/org/opends/server/replication/protocol/StartMsg.java
+++ b/opends/src/server/org/opends/server/replication/protocol/StartMsg.java
@@ -78,10 +78,10 @@
     /* The message header is stored in the form :
      * <message type><protocol version><generation id><group id>
      */
-    builder.append(msgType);
-    builder.append((byte) protocolVersion);
-    builder.appendUTF8(generationId);
-    builder.append(groupId);
+    builder.appendByte(msgType);
+    builder.appendByte((byte) protocolVersion);
+    builder.appendLongUTF8(generationId);
+    builder.appendByte(groupId);
   }
 
   /**
@@ -97,10 +97,10 @@
     /* The message header is stored in the form :
      * <message type><protocol version><generation id>
      */
-    builder.append(msgType);
-    builder.append((byte) ProtocolVersion.REPLICATION_PROTOCOL_V1_REAL);
-    builder.append((byte) 0);
-    builder.appendUTF8(generationId);
+    builder.appendByte(msgType);
+    builder.appendByte((byte) ProtocolVersion.REPLICATION_PROTOCOL_V1_REAL);
+    builder.appendByte((byte) 0);
+    builder.appendLongUTF8(generationId);
   }
 
   /**

--
Gitblit v1.10.0