Removed calls to toString() when calling a logger method. Also removed funn...

Jean-Noel Rouvignac

31.45.2014 d2bbac5c73611fbf1d2c9a767e1b9ad910996747

 opendj3-server-dev/src/server/org/opends/server/tools/upgrade/FileManager.java

@@ -186,8 +186,7 @@
    }
    else
    {
      logger.debug(LocalizableMessage.raw(new StringBuilder("File '").append(file.toString())
          .append("' does not exist").toString()));
      logger.debug(LocalizableMessage.raw("File '" + file + "' does not exist"));
    }
  }

@@ -370,9 +369,8 @@
        {
          if (insureParentsExist(destination))
          {
            final LocalizableMessage message = LocalizableMessage.raw("Copying file '%s' to '%s'",
                objectFile.getAbsolutePath(), destination.getAbsolutePath());
            logger.debug(LocalizableMessage.raw(message.toString()));
            logger.debug(LocalizableMessage.raw("Copying file '%s' to '%s'",
                objectFile.getAbsolutePath(), destination.getAbsolutePath()));
            FileInputStream fis = null;
            FileOutputStream fos = null;
            try
@@ -416,17 +414,15 @@
          {
            final LocalizableMessage errMsg = INFO_ERROR_COPYING_FILE.get(
                objectFile.getAbsolutePath(), destination.getAbsolutePath());
            logger.error(LocalizableMessage.raw(errMsg.toString()));
            logger.error(LocalizableMessage.raw(errMsg));
            throw new IOException(errMsg.toString());
          }
        }
        else
        {
          final LocalizableMessage message = LocalizableMessage.raw(
          logger.debug(LocalizableMessage.raw(
              "Ignoring file '%s' since '%s' already exists",
              objectFile.getAbsolutePath(), destination.getAbsolutePath());

          logger.debug(LocalizableMessage.raw(message.toString()));
              objectFile.getAbsolutePath(), destination.getAbsolutePath()));
        }
      }
    }