search

blickly / closure-compiler-issues

0 stars 0 forks source link

@nosideeffect is ignored for Math.floor #89

Closed blickly closed 9 years ago

blickly commented 9 years ago 
What steps will reproduce the problem?
Try compiling Math.floor(0) in advanced mode with the online service.

What is the expected output? What do you see instead?
Expected: "", actual: "Math.floor(0);"

What version of the product are you using? On what operating system?
r60 I guess

Please provide any additional information below.
This change would help reducing the size of the compiled base.js.
Currently "Math.floor(Math.random()*2147483648).toString(36);" is left over.

Original issue reported on code.google.com by pallosp@google.com on 2010-01-11 17:45:25

blickly commented 9 years ago 
compute_function_side_effects will be turned on in the next release is all goes well.

Original issue reported on code.google.com by concavelenz on 2010-01-11 23:54:40

blickly commented 9 years ago 
Now released.

Original issue reported on code.google.com by concavelenz on 2010-02-04 01:00:00