You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2007/10/05 12:01:46 UTC
svn commit: r582185 - in /geronimo/sandbox/gshell/trunk:
gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandNotFoundException.java
gshell-core/src/main/java/org/apache/geronimo/gshell/DefaultCommandExecutor.java
Author: jdillon
Date: Fri Oct 5 03:01:41 2007
New Revision: 582185
URL: http://svn.apache.org/viewvc?rev=582185&view=rev
Log:
LayoutManager will throw an exception
Removed:
geronimo/sandbox/gshell/trunk/gshell-command-api/src/main/java/org/apache/geronimo/gshell/command/CommandNotFoundException.java
Modified:
geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/DefaultCommandExecutor.java
Modified: geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/DefaultCommandExecutor.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/DefaultCommandExecutor.java?rev=582185&r1=582184&r2=582185&view=diff
==============================================================================
--- geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/DefaultCommandExecutor.java (original)
+++ geronimo/sandbox/gshell/trunk/gshell-core/src/main/java/org/apache/geronimo/gshell/DefaultCommandExecutor.java Fri Oct 5 03:01:41 2007
@@ -22,7 +22,6 @@
import org.apache.geronimo.gshell.command.Command;
import org.apache.geronimo.gshell.command.CommandContext;
import org.apache.geronimo.gshell.command.CommandExecutor;
-import org.apache.geronimo.gshell.command.CommandNotFoundException;
import org.apache.geronimo.gshell.command.IO;
import org.apache.geronimo.gshell.command.Variables;
import org.apache.geronimo.gshell.common.Arguments;
@@ -96,11 +95,8 @@
log.info("Executing ({}): [{}]", path, Arguments.asString(args));
- // Look up the command descriptor for the given path
- final Command command = layoutManager.find(path);
- if (command == null) {
- throw new CommandNotFoundException(path);
- }
+ // Look up the command for the given path
+ Command command = layoutManager.find(path);
// Setup the command context and pass it to the command instance
CommandContext context = new CommandContext() {
@@ -114,7 +110,6 @@
public Variables getVariables() {
return vars;
}
-
};
// Setup command timings