Fixes jashkenas#1467. Catch now introduces its parameter to scope.

zmthy · michaelficarra · commit 4fc52cd08eaf · 2011-07-06T22:15:35.000-04:00
diff --git a/lib/nodes.js b/lib/nodes.js
diff --git a/src/nodes.coffee b/src/nodes.coffee
@@ -1402,6 +1402,7 @@ exports.Try = class Try extends Base
     o.indent  += TAB
     errorPart = if @error then " (#{ @error.compile o }) " else ' '
     catchPart = if @recovery
+      o.scope.add @error.value, 'param'
       " catch#{errorPart}{\n#{ @recovery.compile o, LEVEL_TOP }\n#{@tab}}"
     else unless @ensure or @recovery
       ' catch (_e) {}'
diff --git a/test/scope.coffee b/test/scope.coffee
@@ -26,3 +26,9 @@ test "siblings of variadic arguments shouldn't break out.", ->
   oops = (x,args...) ->
   oops(20, 1,2,3)
   eq x, 10
+
+test "catch statements should introduce their argument to scope", ->
+  try throw ''
+  catch e
+    do -> e = 5
+    eq 5, e
