On Tue, Aug 10, 2010 at 1:09 PM, joegg <joega...@gmail.com> wrote:
> diff --git a/src/jvm/clojure/lang/Compiler.java b/src/jvm/clojure/lang/
> Compiler.java
> index f5684f1..af55660 100644
> --- a/src/jvm/clojure/lang/Compiler.java
> +++ b/src/jvm/clojure/lang/Compiler.java
> @@ -1775,7 +1775,7 @@ public static class TryExpr implements Expr{
> gen.visitTryCatchBlock(startTry, endTry,
> clause.label, clause.c.getName().replace('.', '/'));
> }
> if(finallyExpr != null)
> - gen.visitTryCatchBlock(startTry, endTryCatch,
> finallyLabel, null);
> + gen.visitTryCatchBlock(startTry, endTry,
> finallyLabel, null);
> for(int i = 0; i < catchExprs.count(); i++)
> {
> CatchClause clause = (CatchClause)
> catchExprs.nth(i);
>
>
> This fixes the behavior we're seeing, but, ummm... might break other
> things, I suppose. The tests I've written don't cover all the
> expected behavior of try/catch/finally.
That patch seems to essentially reverse this one:
http://github.com/clojure/clojure/commit/5e9f2b293b307aa7953cd390360d24549e542b92
...which suggests to me there must be a better solution, though I
don't see yet what it would be.
--Chouser
--
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
