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/StartECLSessionMsg.java | 18 +++++++++---------
1 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/opends/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java b/opends/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java
index 694c2f5..d62c9e9 100644
--- a/opends/src/server/org/opends/server/replication/protocol/StartECLSessionMsg.java
+++ b/opends/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(StaticUtils.collectionToString(excludedBaseDNs, ";"));
+ builder.appendIntUTF8((int) firstChangeNumber);
+ builder.appendIntUTF8((int) lastChangeNumber);
+ builder.appendCSNUTF8(csn);
+ builder.appendIntUTF8(isPersistent.ordinal());
+ builder.appendString(crossDomainServerState);
+ builder.appendString(operationId);
+ builder.appendString(StaticUtils.collectionToString(excludedBaseDNs, ";"));
return builder.toByteArray();
}
--
Gitblit v1.10.0