From a1f030edc5d19b575470f0d79033d9f7471ab172 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Fri, 31 Jan 2014 14:49:58 +0000
Subject: [PATCH] Removed now useless calls to .toString() on arguments of ArgN.get(...).

---
 opendj3-server-dev/src/server/org/opends/server/tools/LDAPModify.java |   23 ++++++++---------------
 1 files changed, 8 insertions(+), 15 deletions(-)

diff --git a/opendj3-server-dev/src/server/org/opends/server/tools/LDAPModify.java b/opendj3-server-dev/src/server/org/opends/server/tools/LDAPModify.java
index 0c218e4..7f0e21c 100644
--- a/opendj3-server-dev/src/server/org/opends/server/tools/LDAPModify.java
+++ b/opendj3-server-dev/src/server/org/opends/server/tools/LDAPModify.java
@@ -263,14 +263,12 @@
             attributes.add(new LDAPAttribute(a));
           }
           protocolOp = new AddRequestProtocolOp(asn1OctetStr, attributes);
-          out.println(INFO_PROCESSING_OPERATION.get(
-                  operationType, asn1OctetStr.toString()));
+          out.println(INFO_PROCESSING_OPERATION.get(operationType, asn1OctetStr));
           break;
         case DELETE:
           operationType = "DELETE";
           protocolOp = new DeleteRequestProtocolOp(asn1OctetStr);
-          out.println(INFO_PROCESSING_OPERATION.get(
-                  operationType, asn1OctetStr.toString()));
+          out.println(INFO_PROCESSING_OPERATION.get(operationType, asn1OctetStr));
           break;
         case MODIFY:
           operationType = "MODIFY";
@@ -278,8 +276,7 @@
           ArrayList<RawModification> mods =
             new ArrayList<RawModification>(modEntry.getModifications());
           protocolOp = new ModifyRequestProtocolOp(asn1OctetStr, mods);
-          out.println(INFO_PROCESSING_OPERATION.get(
-                  operationType, asn1OctetStr.toString()));
+          out.println(INFO_PROCESSING_OPERATION.get(operationType, asn1OctetStr));
           break;
         case MODIFY_DN:
           operationType = "MODIFY DN";
@@ -299,8 +296,7 @@
                  modDNEntry.deleteOldRDN());
           }
 
-          out.println(INFO_PROCESSING_OPERATION.get(
-                  operationType, asn1OctetStr.toString()));
+          out.println(INFO_PROCESSING_OPERATION.get(operationType, asn1OctetStr));
           break;
         default:
           break;
@@ -415,9 +411,7 @@
           }
         } else
         {
-          LocalizableMessage msg = INFO_OPERATION_SUCCESSFUL.get(
-                  operationType, asn1OctetStr.toString());
-          out.println(msg);
+          out.println(INFO_OPERATION_SUCCESSFUL.get(operationType, asn1OctetStr));
 
           if (errorMessage != null)
           {
@@ -504,14 +498,13 @@
             if(c instanceof LDAPControl)
             {
               // Don't really need to decode since its just an octet string.
-              out.println(INFO_CHANGE_NUMBER_CONTROL_RESULT.get(operationType,
-                  ((LDAPControl)c).getValue().toString()));
+              out.println(INFO_CHANGE_NUMBER_CONTROL_RESULT.get(
+                  operationType, ((LDAPControl)c).getValue()));
             }
             else
             {
               out.println(INFO_CHANGE_NUMBER_CONTROL_RESULT.get(operationType,
-                  ((ChangeNumberControlPlugin.ChangeNumberControl)c).
-                      getCSN().toString()));
+                  ((ChangeNumberControlPlugin.ChangeNumberControl)c).getCSN()));
             }
           }
         }

--
Gitblit v1.10.0