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 2008/09/20 10:57:20 UTC

svn commit: r697336 - /geronimo/gshell/trunk/gshell-api/src/main/java/org/apache/geronimo/gshell/command/CommandResult.java

Author: jdillon
Date: Sat Sep 20 01:57:20 2008
New Revision: 697336

URL: http://svn.apache.org/viewvc?rev=697336&view=rev
Log:
Fixed handling of Throwables

Modified:
    geronimo/gshell/trunk/gshell-api/src/main/java/org/apache/geronimo/gshell/command/CommandResult.java

Modified: geronimo/gshell/trunk/gshell-api/src/main/java/org/apache/geronimo/gshell/command/CommandResult.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-api/src/main/java/org/apache/geronimo/gshell/command/CommandResult.java?rev=697336&r1=697335&r2=697336&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-api/src/main/java/org/apache/geronimo/gshell/command/CommandResult.java (original)
+++ geronimo/gshell/trunk/gshell-api/src/main/java/org/apache/geronimo/gshell/command/CommandResult.java Sat Sep 20 01:57:20 2008
@@ -36,7 +36,7 @@
 
     private final Notification notification;
 
-    private CommandResult(final Object value, final Exception failure, final Notification notification) {
+    private CommandResult(final Object value, final Throwable failure, final Notification notification) {
         this.value = value;
         this.failure = failure;
         this.notification = notification;
@@ -46,10 +46,11 @@
         this(value, null, null);
     }
 
-    public CommandResult(final Exception failure) {
+    public CommandResult(final Throwable failure) {
         this(null, failure, null);
 
         assert failure != null;
+        assert !(failure instanceof Notification);
     }
 
     public CommandResult(final Notification notification) {