BarryCarlyon / minify

Automatically exported from code.google.com/p/minify (personal copy)
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

JSMin munges jQuery-IU 1.8rc3 #168

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Minify version: 2.1.3
PHP version: 5.2.8

What steps will reproduce the problem?
1. I minified and compressed jquery-ui 1.8rc3 (the most recent version as of at 
least a week ago)
2. The script will not run because the minifier removed a space that made an 
increment operand 
invalid. Specifically, it removed from 
"this.cookie=this.options.cookie.name||"ui-tabs-"+ ++d" 
the space in "+ ++d", which made it look like "+++d" to the compiler.

Expected output: "+ ++d"

Actual output: "+++d"

Did any unit tests FAIL? NO

Please provide any additional information below. If this has to do with URI
rewriting, please include your DOCUMENT_ROOT.

I do not rewrite and URIs.

Original issue reported on code.google.com by mjohnso...@gmail.com on 4 Apr 2010 at 5:37

GoogleCodeExporter commented 9 years ago
Is there a pre-minified (with YUICompressor) version you could use? The JSMin 
algorithm chokes on this syntax and there's not a whole lot we can do about it. 
The 
syntax is valid, but doesn't pass JSLint. +(++d) should work.

Original comment by mrclay....@gmail.com on 4 Apr 2010 at 6:25

GoogleCodeExporter commented 9 years ago

Original comment by mrclay....@gmail.com on 8 May 2010 at 7:36