From 34b97b2d33a1196d3b22bd0ade1e758be613f7f3 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Fri, 06 Jun 2014 14:37:42 +0000
Subject: [PATCH] Ensured no accidental protocol break can happen by renaming all ByteArrayBuilder.append*() methods and getting rid of all method overloading.

---
 opendj3-server-dev/src/server/org/opends/server/replication/protocol/UpdateMsg.java |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/UpdateMsg.java b/opendj3-server-dev/src/server/org/opends/server/replication/protocol/UpdateMsg.java
index 317d3ea..3cf7e78 100644
--- a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/UpdateMsg.java
+++ b/opendj3-server-dev/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