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

diff --git a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java b/opendj3-server-dev/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java
index e4e4f1a..76eda35 100644
--- a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java
+++ b/opendj3-server-dev/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java
@@ -198,17 +198,17 @@
   public byte[] getBytes(short protocolVersion)
   {
     final ByteArrayBuilder builder = new ByteArrayBuilder();
-    builder.append(MSG_TYPE_START_ECL_SESSION);
-    builder.appendUTF8(eclRequestType.ordinal());
+    builder.appendByte(MSG_TYPE_START_ECL_SESSION);
+    builder.appendIntUTF8(eclRequestType.ordinal());
     // FIXME JNR Changing the lines below to use long would require a protocol
     // version change. Leave it like this for now until the need arises.
-    builder.appendUTF8((int) firstChangeNumber);
-    builder.appendUTF8((int) lastChangeNumber);
-    builder.appendUTF8(csn);
-    builder.appendUTF8(isPersistent.ordinal());
-    builder.append(crossDomainServerState);
-    builder.append(operationId);
-    builder.append(Utils.joinAsString(";", excludedBaseDNs));
+    builder.appendIntUTF8((int) firstChangeNumber);
+    builder.appendIntUTF8((int) lastChangeNumber);
+    builder.appendCSNUTF8(csn);
+    builder.appendIntUTF8(isPersistent.ordinal());
+    builder.appendString(crossDomainServerState);
+    builder.appendString(operationId);
+    builder.appendString(Utils.joinAsString(";", excludedBaseDNs));
     return builder.toByteArray();
   }
 

--
Gitblit v1.10.0