github/coffeescript
1
1
Fork 0
mirror of https://github.com/jashkenas/coffeescript.git synced 2026-02-18 11:31:20 -05:00
Code Issues Packages Projects Releases Wiki Activity

removing no_paren -- it was optimizing away order of operations

Browse Source
This commit is contained in:
Jeremy Ashkenas
2009-12-26 11:55:34 -08:00
parent 08dddb27a0
commit 694833dbd0
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
lib/coffee_script/nodes.rb
View File

@@ -207,7 +207,7 @@ module CoffeeScript
def compile(o={}) def compile(o={})
o = super(o) o = super(o)
args = @arguments.map{|a| a.compile(o.merge(:no_paren => true)) }.join(', ') args = @arguments.map{|a| a.compile(o) }.join(', ')
return write(compile_super(args, o)) if super? return write(compile_super(args, o)) if super?
prefix = @new ? "new " : '' prefix = @new ? "new " : ''
write("#{prefix}#{@variable.compile(o)}(#{args})") write("#{prefix}#{@variable.compile(o)}(#{args})")
@@ -511,7 +511,7 @@ module CoffeeScript
o = super(o) o = super(o)
o.delete(:return) o.delete(:return)
indent = o[:indent] + TAB indent = o[:indent] + TAB
cond = @condition.compile(o.merge(:no_paren => true)) cond = @condition.compile(o)
write("while (#{cond}) {\n#{@body.compile(o.merge(:indent => indent))}\n#{o[:indent]}}") write("while (#{cond}) {\n#{@body.compile(o.merge(:indent => indent))}\n#{o[:indent]}}")
end end
end end
@@ -656,7 +656,7 @@ module CoffeeScript
o = super(o) o = super(o)
compiled = @expressions.compile(o) compiled = @expressions.compile(o)
compiled = compiled[0...-1] if compiled[-1..-1] == ';' compiled = compiled[0...-1] if compiled[-1..-1] == ';'
write(o[:no_paren] ? compiled : "(#{compiled})") write("(#{compiled})")
end end
end end