2011/1/30 Brion Vibber <[email protected]>:
> I noticed a regression in the JS minification with a bit of sloppy
> third-party code being imported through ResourceLoader in SVGEdit:
>
> https://bugzilla.wikimedia.org/show_bug.cgi?id=27046
>
Fixed now. The regex stripping C++-style // comments was also
stripping the newline following them. I've now changed it to replace a
comment plus newline (i.e. !//[^\r\n]*[\r\n]!) with a newline, so
these newlines are always preserved, regardless of whether vertical
whitespace stripping and collapsing is enabled.

> I can work around it in the extension, but we should watch out and make sure
> we've got regression tests covering any cases we find.
>
Yes, we need minifier tests.

Roan Kattouw (Catrope)

_______________________________________________
Wikitech-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to