Roan Kattouw <> changed:

           What    |Removed                     |Added
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Roan Kattouw <> 2011-01-31 12:48:56 
UTC ---
(In reply to comment #0)
> The combination of these two seems to confuse the new minifier, and it ends up
> slapping them together with neither a semicolon nor any whitespace at all!
The real bug here is that the newline after a C++-style // comment is stripped
unconditionally, even though the minifier keeps each statement on its own line
unless $wgResourceLoaderMinifyJSVerticalSpace is set to true. I have changed it
to unconditionally *preserve* such newlines in r81246, and confirmed that this
fixes your test case.

> Adding to MW 1.17 blockers (unless we won't be shipping this version of the
> minifier -- the old code did fine)
The fix has been tagged for 1.17 merging.

Configure bugmail:
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Wikibugs-l mailing list

Reply via email to