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/UpdateMsg.java |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/opends/src/server/org/opends/server/replication/protocol/UpdateMsg.java b/opends/src/server/org/opends/server/replication/protocol/UpdateMsg.java
index 28febb3..04ac885 100644
--- a/opends/src/server/org/opends/server/replication/protocol/UpdateMsg.java
+++ b/opends/src/server/org/opends/server/replication/protocol/UpdateMsg.java
@@ -220,12 +220,12 @@
   {
     final ByteArrayBuilder builder =
         new ByteArrayBuilder(bytes(6) + csnsUTF8(1));
-    builder.append(msgType);
-    builder.append((byte) ProtocolVersion.getCurrentVersion());
-    builder.appendUTF8(getCSN());
-    builder.append(assuredFlag);
-    builder.append(assuredMode.getValue());
-    builder.append(safeDataLevel);
+    builder.appendByte(msgType);
+    builder.appendByte((byte) ProtocolVersion.getCurrentVersion());
+    builder.appendCSNUTF8(getCSN());
+    builder.appendBoolean(assuredFlag);
+    builder.appendByte(assuredMode.getValue());
+    builder.appendByte(safeDataLevel);
     return builder;
   }
 
@@ -281,7 +281,7 @@
   {
     final ByteArrayBuilder builder = encodeHeader(MSG_TYPE_GENERIC_UPDATE,
         ProtocolVersion.getCurrentVersion());
-    builder.append(payload);
+    builder.appendByteArray(payload);
     return builder.toByteArray();
   }
 

--
Gitblit v1.10.0