From 31216400c324b43c15b8a9eea6d89604247ebb14 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Mon, 03 Feb 2014 12:10:05 +0000
Subject: [PATCH] Removed now useless calls to String.valueOf() on arguments of ArgN.get(...).

---
 opendj3-server-dev/src/server/org/opends/server/schema/ObjectClassSyntax.java |   50 +++++++++++++++++++-------------------------------
 1 files changed, 19 insertions(+), 31 deletions(-)

diff --git a/opendj3-server-dev/src/server/org/opends/server/schema/ObjectClassSyntax.java b/opendj3-server-dev/src/server/org/opends/server/schema/ObjectClassSyntax.java
index c8c3042..a6995e5 100644
--- a/opendj3-server-dev/src/server/org/opends/server/schema/ObjectClassSyntax.java
+++ b/opendj3-server-dev/src/server/org/opends/server/schema/ObjectClassSyntax.java
@@ -282,9 +282,8 @@
     if (c != '(')
     {
       LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_EXPECTED_OPEN_PARENTHESIS.
-          get(valueStr, (pos-1), String.valueOf(c));
-      throw new DirectoryException(
-              ResultCode.INVALID_ATTRIBUTE_SYNTAX, message);
+          get(valueStr, (pos-1), c);
+      throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX, message);
     }
 
 
@@ -337,9 +336,8 @@
           // This must have been an illegal character.
           LocalizableMessage message =
               ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR_IN_NUMERIC_OID.
-                get(valueStr, String.valueOf(c), (pos-1));
-          throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
-                                       message);
+                get(valueStr, c, (pos-1));
+          throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX, message);
         }
         else
         {
@@ -363,7 +361,7 @@
           // This must have been an illegal character.
           LocalizableMessage message =
               ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR_IN_STRING_OID.
-                get(valueStr, String.valueOf(c), (pos-1));
+                get(valueStr, c, (pos-1));
           throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                        message);
         }
@@ -496,8 +494,7 @@
         else
         {
           // This is an illegal character.
-          LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(
-              valueStr, String.valueOf(c), (pos-1));
+          LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(valueStr, c, (pos-1));
           throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                        message);
         }
@@ -605,10 +602,8 @@
                 // This is bad because we don't know what the superior oc
                 // is so we can't base this objectclass on it.
                 LocalizableMessage message =
-                    WARN_ATTR_SYNTAX_OBJECTCLASS_UNKNOWN_SUPERIOR_CLASS.
-                      get(String.valueOf(oid), String.valueOf(woidBuffer));
-                throw new DirectoryException(ResultCode.CONSTRAINT_VIOLATION,
-                                             message);
+                    WARN_ATTR_SYNTAX_OBJECTCLASS_UNKNOWN_SUPERIOR_CLASS.get(oid, woidBuffer);
+                throw new DirectoryException(ResultCode.CONSTRAINT_VIOLATION, message);
               }
             }
 
@@ -623,8 +618,7 @@
             }
             else if (c != '$')
             {
-              LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(
-                  valueStr, String.valueOf(c), (pos-1));
+              LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(valueStr, c, (pos-1));
               throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                            message);
             }
@@ -648,10 +642,8 @@
               // This is bad because we don't know what the superior oc
               // is so we can't base this objectclass on it.
               LocalizableMessage message =
-                  WARN_ATTR_SYNTAX_OBJECTCLASS_UNKNOWN_SUPERIOR_CLASS.
-                    get(String.valueOf(oid), String.valueOf(woidBuffer));
-              throw new DirectoryException(ResultCode.CONSTRAINT_VIOLATION,
-                                           message);
+                  WARN_ATTR_SYNTAX_OBJECTCLASS_UNKNOWN_SUPERIOR_CLASS.get(oid, woidBuffer);
+              throw new DirectoryException(ResultCode.CONSTRAINT_VIOLATION, message);
             }
           }
           listSupOCs.add(superiorClass);
@@ -726,8 +718,7 @@
             }
             else if (c != '$')
             {
-              LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(
-                  valueStr, String.valueOf(c), (pos-1));
+              LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(valueStr, c, (pos-1));
               throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                            message);
             }
@@ -812,8 +803,7 @@
             }
             else if (c != '$')
             {
-              LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(
-                  valueStr, String.valueOf(c), (pos-1));
+              LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(valueStr, c, (pos-1));
               throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                            message);
             }
@@ -1040,7 +1030,7 @@
     if (c != '\'')
     {
       LocalizableMessage message = WARN_ATTR_SYNTAX_OBJECTCLASS_EXPECTED_QUOTE_AT_POS.get(
-          valueStr, startPos, String.valueOf(c));
+          valueStr,startPos, c);
       throw new DirectoryException(
               ResultCode.INVALID_ATTRIBUTE_SYNTAX, message);
     }
@@ -1127,7 +1117,7 @@
     if (c != '\'')
     {
       LocalizableMessage message = WARN_ATTR_SYNTAX_OBJECTCLASS_EXPECTED_QUOTE_AT_POS.get(
-          valueStr, startPos, String.valueOf(c));
+          valueStr,startPos, c);
       throw new DirectoryException(
               ResultCode.INVALID_ATTRIBUTE_SYNTAX, message);
     }
@@ -1248,7 +1238,7 @@
           // This must have been an illegal character.
           LocalizableMessage message =
             ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR_IN_NUMERIC_OID.
-                get(lowerStr, String.valueOf(c), (startPos-1));
+                get(lowerStr, c, (startPos-1));
           throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                        message);
         }
@@ -1286,7 +1276,7 @@
           // This must have been an illegal character.
           LocalizableMessage message =
             ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR_IN_STRING_OID.
-                get(lowerStr, String.valueOf(c), (startPos-1));
+                get(lowerStr, c, (startPos-1));
           throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                        message);
         }
@@ -1295,8 +1285,7 @@
     else
     {
       LocalizableMessage message =
-          ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(
-                  lowerStr, String.valueOf(c), startPos);
+          ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(lowerStr, c, startPos);
       throw new DirectoryException(
               ResultCode.INVALID_ATTRIBUTE_SYNTAX, message);
     }
@@ -1406,8 +1395,7 @@
         else if (c == '(')
         {
           // This is an illegal character.
-          LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(
-              valueStr, String.valueOf(c), startPos);
+          LocalizableMessage message = ERR_ATTR_SYNTAX_OBJECTCLASS_ILLEGAL_CHAR.get(valueStr, c, startPos);
           throw new DirectoryException(ResultCode.INVALID_ATTRIBUTE_SYNTAX,
                                        message);
         }

--
Gitblit v1.10.0