Issue #870 ... placeholders in destructuring assignment.

jashkenas · Jan 31, 2011 · 4ce374b · 4ce374b
1 parent 8b8e8a9
commit 4ce374b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/nodes.js b/lib/nodes.js
diff --git a/src/nodes.coffee b/src/nodes.coffee
@@ -908,6 +908,7 @@ exports.Assign = class Assign extends Base
     {value}   = this
     {objects} = @variable.base
     unless olen = objects.length
+      return false if top
       code = value.compile o
       return if o.level >= LEVEL_OP then "(#{code})" else code
     isObject = @variable.isObject()
@@ -968,7 +969,7 @@ exports.Assign = class Assign extends Base
         val = new Value new Literal(vvar), [new (if acc then Access else Index) idx]
       assigns.push new Assign(obj, val, null, param: @param).compile o, LEVEL_TOP
     assigns.push vvar unless top
-    code = assigns.join ', '
+    code = (compact assigns).join ', '
     if o.level < LEVEL_LIST then code else "(#{code})"
 
   # When compiling a conditional assignment, take care to ensure that the