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/AckMsg.java | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/AckMsg.java b/opendj3-server-dev/src/server/org/opends/server/replication/protocol/AckMsg.java
index 853d589..128386f 100644
--- a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/AckMsg.java
+++ b/opendj3-server-dev/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