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

diff --git a/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/LDAPUpdateMsg.java b/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/LDAPUpdateMsg.java
index a47d015..43cb522 100644
--- a/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/LDAPUpdateMsg.java
+++ b/opendj-sdk/opends/src/server/org/opends/server/replication/protocol/LDAPUpdateMsg.java
@@ -216,14 +216,14 @@
      * <assured mode> <safe data level>
      */
     final ByteArrayBuilder builder = new ByteArrayBuilder();
-    builder.append(msgType);
-    builder.append((byte) protocolVersion);
-    builder.appendUTF8(csn);
-    builder.append(dn);
-    builder.append(entryUUID);
-    builder.append(assuredFlag);
-    builder.append(assuredMode.getValue());
-    builder.append(safeDataLevel);
+    builder.appendByte(msgType);
+    builder.appendByte((byte) protocolVersion);
+    builder.appendCSNUTF8(csn);
+    builder.appendDN(dn);
+    builder.appendString(entryUUID);
+    builder.appendBoolean(assuredFlag);
+    builder.appendByte(assuredMode.getValue());
+    builder.appendByte(safeDataLevel);
     return builder;
   }
 
@@ -240,11 +240,11 @@
      * <operation type><CSN><dn><assured><entryuuid><change>
      */
     final ByteArrayBuilder builder = new ByteArrayBuilder();
-    builder.append(msgType);
-    builder.appendUTF8(csn);
-    builder.append(assuredFlag);
-    builder.append(dn);
-    builder.append(entryUUID);
+    builder.appendByte(msgType);
+    builder.appendCSNUTF8(csn);
+    builder.appendBoolean(assuredFlag);
+    builder.appendDN(dn);
+    builder.appendString(entryUUID);
     return builder;
   }
 

--
Gitblit v1.10.0