From e8cead474d5ce2b933d931f0c4743a78e68d9cfc Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Tue, 18 Aug 2015 09:39:52 +0000
Subject: [PATCH] Replaced uses of StringBuilder.append(Utils.joinAsString(String, Collection)) with the more efficient Utils.joinAsString(StringBuilder, String, Collection).
---
opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/BindResponseProtocolOp.java | 70 +++++-----------------------------
1 files changed, 11 insertions(+), 59 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/BindResponseProtocolOp.java b/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/BindResponseProtocolOp.java
index 3462cb5..800ffd9 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/BindResponseProtocolOp.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/BindResponseProtocolOp.java
@@ -25,17 +25,17 @@
* Portions Copyright 2014-2015 ForgeRock AS
*/
package org.opends.server.protocols.ldap;
-import org.forgerock.i18n.LocalizableMessage;
-import java.util.List;
import java.io.IOException;
+import java.util.List;
-import org.forgerock.opendj.io.*;
-import org.opends.server.types.DN;
+import org.forgerock.i18n.LocalizableMessage;
+import org.forgerock.i18n.slf4j.LocalizedLogger;
+import org.forgerock.opendj.io.ASN1Writer;
import org.forgerock.opendj.ldap.ByteString;
import org.forgerock.util.Utils;
+import org.opends.server.types.DN;
-import org.forgerock.i18n.slf4j.LocalizedLogger;
import static org.opends.server.protocols.ldap.LDAPConstants.*;
import static org.opends.server.util.ServerConstants.*;
@@ -54,13 +54,10 @@
/** The matched DN for this response. */
private DN matchedDN;
-
/** The result code for this response. */
private int resultCode;
-
/** The set of referral URLs for this response. */
private List<String> referralURLs;
-
/** The error message for this response. */
private LocalizableMessage errorMessage;
@@ -74,11 +71,6 @@
public BindResponseProtocolOp(int resultCode)
{
this.resultCode = resultCode;
-
- errorMessage = null;
- matchedDN = null;
- referralURLs = null;
- serverSASLCredentials = null;
}
@@ -94,10 +86,6 @@
{
this.resultCode = resultCode;
this.errorMessage = errorMessage;
-
- matchedDN = null;
- referralURLs = null;
- serverSASLCredentials = null;
}
@@ -117,8 +105,6 @@
this.errorMessage = errorMessage;
this.matchedDN = matchedDN;
this.referralURLs = referralURLs;
-
- serverSASLCredentials = null;
}
@@ -207,36 +193,19 @@
return serverSASLCredentials;
}
-
-
- /**
- * Retrieves the BER type for this protocol op.
- *
- * @return The BER type for this protocol op.
- */
+ @Override
public byte getType()
{
return OP_TYPE_BIND_RESPONSE;
}
-
-
- /**
- * Retrieves the name for this protocol op type.
- *
- * @return The name for this protocol op type.
- */
+ @Override
public String getProtocolOpName()
{
return "Bind Response";
}
- /**
- * Writes this protocol op to an ASN.1 output stream.
- *
- * @param stream The ASN.1 output stream to write to.
- * @throws IOException If a problem occurs while writing to the stream.
- */
+ @Override
public void write(ASN1Writer stream) throws IOException
{
stream.writeStartSequence(OP_TYPE_BIND_RESPONSE);
@@ -279,14 +248,7 @@
stream.writeEndSequence();
}
-
-
- /**
- * Appends a string representation of this LDAP protocol op to the provided
- * buffer.
- *
- * @param buffer The buffer to which the string should be appended.
- */
+ @Override
public void toString(StringBuilder buffer)
{
buffer.append("BindResponse(resultCode=");
@@ -305,7 +267,7 @@
if (referralURLs != null && !referralURLs.isEmpty())
{
buffer.append(", referralURLs={");
- buffer.append(Utils.joinAsString(", ", referralURLs));
+ Utils.joinAsString(buffer, ", ", referralURLs);
buffer.append("}");
}
@@ -318,16 +280,7 @@
buffer.append(")");
}
-
-
- /**
- * Appends a multi-line string representation of this LDAP protocol op to the
- * provided buffer.
- *
- * @param buffer The buffer to which the information should be appended.
- * @param indent The number of spaces from the margin that the lines should
- * be indented.
- */
+ @Override
public void toString(StringBuilder buffer, int indent)
{
StringBuilder indentBuf = new StringBuilder(indent);
@@ -386,4 +339,3 @@
}
}
}
-
--
Gitblit v1.10.0