Hi Mark,

On Fri, Sep 18, 2009 at 3:43 PM, Mark Evenson mevenson@common-lisp.net wrote:

Author: mevenson Date: Fri Sep 18 09:43:42 2009 New Revision: 12153

Log: Tenative fix for #63: transform caught Go throwable to LispError.

Normalized printStackTrace() calls through Debug.trace() interface.

Modified:   trunk/abcl/src/org/armedbear/lisp/Lisp.java

Modified: trunk/abcl/src/org/armedbear/lisp/Lisp.java

--- trunk/abcl/src/org/armedbear/lisp/Lisp.java (original) +++ trunk/abcl/src/org/armedbear/lisp/Lisp.java Fri Sep 18 09:43:42 2009 @@ -123,7 +123,7 @@       }     catch (Throwable t)       {

• t.printStackTrace();

• Debug.trace(t);

}   }

@@ -277,7 +277,7 @@           }         catch (Go go)           {

• throw go;

• return error(go.getCondition());

}         catch (Throw t)           {

This hunk causes "havoc" with the command line interface where people select the "go to toplevel" restart.

I think this is the major change in this commit? If I want to revert this bit, do I have to revert the rest too, or are those just improvements on what was already there?

BTW: With my recent changes, I don't think there's anything wrong with re-throwing the Go anymore: we check for a catching environment before throwing now.

@@ -1042,6 +1042,7 @@       }     catch (Throwable t)       {

• Debug.trace(t);

return null;       }   } @@ -2586,7 +2587,7 @@       }     catch (ClassNotFoundException e)       {

• e.printStackTrace();

• Debug.trace(e);

}   }

---

armedbear-cvs mailing list armedbear-cvs@common-lisp.net http://common-lisp.net/cgi-bin/mailman/listinfo/armedbear-cvs