[DL-Learner SVN] SF.net SVN: dl-learner: [570] trunk/src/dl-learner/org/dllearner/cli

[<jen...@us...>]

Revision: 570
          http://dl-learner.svn.sourceforge.net/dl-learner/?rev=570&view=rev
Author:   jenslehmann
Date:     2008-02-13 09:32:23 -0800 (Wed, 13 Feb 2008)

Log Message:
-----------
fixed bug #1892730

Modified Paths:
--------------
    trunk/src/dl-learner/org/dllearner/cli/ConfFileOption.java
    trunk/src/dl-learner/org/dllearner/cli/Start.java

Modified: trunk/src/dl-learner/org/dllearner/cli/ConfFileOption.java
===================================================================
--- trunk/src/dl-learner/org/dllearner/cli/ConfFileOption.java	2008-02-13 14:54:37 UTC (rev 569)
+++ trunk/src/dl-learner/org/dllearner/cli/ConfFileOption.java	2008-02-13 17:32:23 UTC (rev 570)
@@ -81,7 +81,7 @@
 		this.option = option;
 		this.subOption = subOption;
 		doubleValue = value;
-		isDoubleOption = false;
+		isDoubleOption = true;
 	}
 	
 	public ConfFileOption(String option, Set<String> values) {

Modified: trunk/src/dl-learner/org/dllearner/cli/Start.java
===================================================================
--- trunk/src/dl-learner/org/dllearner/cli/Start.java	2008-02-13 14:54:37 UTC (rev 569)
+++ trunk/src/dl-learner/org/dllearner/cli/Start.java	2008-02-13 17:32:23 UTC (rev 570)
@@ -130,12 +130,13 @@
 		String baseDir = file.getParentFile().getPath();		
 		
 		// create component manager instance
-		System.out.print("starting component manager ... ");
+		String message = "starting component manager ... ";
 		long cmStartTime = System.nanoTime();
 		ComponentManager cm = ComponentManager.getInstance();
 		long cmTime = System.nanoTime() - cmStartTime;
-		System.out.println("OK (" + Helper.prettyPrintNanoSeconds(cmTime) + ")");
-
+		message += "OK (" + Helper.prettyPrintNanoSeconds(cmTime) + ")";
+		logger.info(message);
+		
 		// create a mapping between components and prefixes in the conf file
 		Map<Class<? extends Component>, String> componentPrefixMapping = createComponentPrefixMapping();
 
@@ -355,7 +356,7 @@
 					cm.applyConfigEntry(component, entry);
 
 				} else {
-					handleError("The type of conf file entry " + option + " is not correct.");
+					handleError("The type of conf file entry \"" + option.getFullName() + "\" is not correct: value \"" + option.getValue() + "\" not valid for option type \"" + configOption.getClass().getName() + "\".");
 				}
 
 			} catch (InvalidConfigOptionValueException e) {
@@ -702,7 +703,7 @@
 	}
 
 	private static void handleError(String message) {
-		System.err.println(message);
+		logger.error(message);
 		System.exit(0);
 	}
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.