diff --git a/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/Logger.java b/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/Logger.java
index b2c42365533..93a91188a8c 100644
--- a/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/Logger.java
+++ b/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/Logger.java
@@ -131,8 +131,9 @@ public class Logger implements IPropertyChangeListener {
 
 	/** 
 	 * Log an Error message with an exception. Note that the message should already 
-	 * be localized to proper local.<br>
-	 * ie: Resource.getString() should already have been called
+	 * be localized to proper locale.
+	 * @param message the message to log.
+	 * @param ex the exception that caused the condition, may be null.
 	 */
 	public synchronized void logError(String message, Throwable ex) {
 		if (debug_level >= IRemoteSystemsLogging.LOG_ERROR) {
diff --git a/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/LoggerFactory.java b/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/LoggerFactory.java
index 8a3fc3ae2b6..e2ad3124314 100644
--- a/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/LoggerFactory.java
+++ b/rse/plugins/org.eclipse.rse.logging/src/org/eclipse/rse/logging/LoggerFactory.java
@@ -33,6 +33,8 @@ public class LoggerFactory {
 	/**
 	 * Returns the Logger instance for a given plugin. There is only
 	 * one instance of the Logger class per plugin. 
+	 * @param plugin the plugin for which to find or create the log
+	 * @return the logger for that plugin
 	 */
 	public static Logger getLogger(Plugin plugin) {
 		Logger logger = (Logger) pluginTable.get(plugin);
@@ -44,8 +46,9 @@ public class LoggerFactory {
 	}
 
 	/**
-	 * Frees resources used by the Logger instance for the given plugin.<br>
-	 * This methods must be called as part of the the plugin shutdown life cycle.
+	 * Frees resources used by the Logger instance for the given plugin.
+	 * This method must be called as part of the the plugin shutdown life cycle.
+	 * @param plugin the plugin for which to free logging resources
 	 */
 	public static void freeLogger(Plugin plugin) {
 		Logger logger = (Logger) pluginTable.get(plugin);