From 73a7f0d0c650739c667dc965f6ed2cd34756bd1e Mon Sep 17 00:00:00 2001
From: Ludovic Poitou <ludovic.poitou@forgerock.com>
Date: Fri, 23 Aug 2013 13:25:33 +0000
Subject: [PATCH] Small cleanup of use of maps, lists and also few typos in comments, and constants in the QuickSetup. Side effect of renaming a constant, the ReplicationCliMain was touched, and I run more cleanup on it.
---
opends/src/quicksetup/org/opends/quicksetup/ui/QuickSetup.java | 19 +++++++++++--------
1 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/opends/src/quicksetup/org/opends/quicksetup/ui/QuickSetup.java b/opends/src/quicksetup/org/opends/quicksetup/ui/QuickSetup.java
index ecd21da..256000a 100644
--- a/opends/src/quicksetup/org/opends/quicksetup/ui/QuickSetup.java
+++ b/opends/src/quicksetup/org/opends/quicksetup/ui/QuickSetup.java
@@ -23,7 +23,7 @@
*
*
* Copyright 2006-2010 Sun Microsystems, Inc.
- * Portions Copyright 2011 ForgeRock AS
+ * Portions Copyright 2011-2013 ForgeRock AS
*/
package org.opends.quicksetup.ui;
@@ -47,6 +47,7 @@
import java.awt.Cursor;
import java.util.ArrayList;
+import java.util.List;
import java.util.logging.Logger;
import java.util.logging.Level;
import java.util.logging.Handler;
@@ -137,7 +138,7 @@
}
/* In the calls to setCurrentStep the dialog will be created */
- setCurrentStep(application.getFirstWizardStep());
+ setCurrentStep(application.getFirstWizardStep());
}
/**
@@ -404,7 +405,7 @@
}
ProcessBuilder pb;
if (isMacOS()) {
- ArrayList<String> cmd = new ArrayList<String>();
+ List<String> cmd = new ArrayList<String>();
cmd.add(MAC_APPLICATIONS_OPENER);
cmd.add(getScriptPath(
getPath(installation.getControlPanelCommandFile())));
@@ -433,7 +434,7 @@
{
returnValue = process.exitValue();
}
- catch (IllegalThreadStateException ithse)
+ catch (IllegalThreadStateException e)
{
// The process has not exited: assume that the status panel could
// be launched successfully.
@@ -684,10 +685,12 @@
{
progressDetails.append(additionalDetails);
}
- // Note: progressDetails might have a certain number of characters that
- // break Message Formatter (for instance percentages). When fix for
- // issue 2142 was committed it broke this code. So here
- // we use Message.raw instead of calling directly progressDetails.toMessage
+ /*
+ Note: progressDetails might have a certain number of characters that
+ break Message Formatter (for instance percentages). When fix for
+ issue 2142 was committed it broke this code. So here
+ we use Message.raw instead of calling directly progressDetails.toMessage
+ */
return new ProgressDescriptor(status, ratio, newProgressLabel,
Message.raw(progressDetails.toString()));
}
--
Gitblit v1.10.0