From a1afc5123cc95ba841f0ac3d75f0d1d9d75928dd 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.

---
 opendj-sdk/opends/src/server/org/opends/server/replication/protocol/AckMsg.java |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/AckMsg.java b/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/AckMsg.java
index 853d589..128386f 100644
--- a/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/AckMsg.java
+++ b/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/AckMsg.java
@@ -201,14 +201,14 @@
      * error><failed server ids>
      */
     final ByteArrayBuilder builder = new ByteArrayBuilder();
-    builder.append(MSG_TYPE_ACK);
-    builder.appendUTF8(csn);
-    builder.append(hasTimeout);
-    builder.append(hasWrongStatus);
-    builder.append(hasReplayError);
+    builder.appendByte(MSG_TYPE_ACK);
+    builder.appendCSNUTF8(csn);
+    builder.appendBoolean(hasTimeout);
+    builder.appendBoolean(hasWrongStatus);
+    builder.appendBoolean(hasReplayError);
     for (int serverId : failedServers)
     {
-      builder.appendUTF8(serverId);
+      builder.appendIntUTF8(serverId);
     }
     return builder.toByteArray();
   }

--
Gitblit v1.10.0