Commit graph

13 commits

Author SHA1 Message Date
Roan Kattouw
c1ba885db2 Fix a bug in string recognition in JSDistiller. This *really* fixes bug 27481 2011-02-17 22:36:11 +00:00
Roan Kattouw
12407cea19 Fix for r82344, which causes JSDistiller to stop minifying comments because it thought they were regexes. Make the regex regex a little bit stricter again, forbidding a regex from starting with * and other invalid things that were disallowed prior to r82344 (but still dropping / ) and forbidding multiline regexes 2011-02-17 22:07:18 +00:00
Roan Kattouw
e7a2f04176 Revert r82340, caused something linke bug 27481 where spaces were being wrongly collapsed. Fixed by changing regex to be more flexible in protecting regexes; nothing wrong with being overinclusive in what we're protecting 2011-02-17 19:19:50 +00:00
Roan Kattouw
e519eadc99 (bug 27492) Fix regexes for matching JS regexes which could get messed up by the minifier in extreme circumstances 2011-02-17 17:08:06 +00:00
Mark A. Hershberger
b4cee86beb w/s fixups 2011-02-12 04:06:22 +00:00
Roan Kattouw
158c26e49b Reimplement r80900, was lost when code was moved around 2011-02-08 10:26:30 +00:00
Roan Kattouw
3d6c28a9db (bug 27046) Do not strip newlines following C++-style // comments. Prior to this commit, such newlines were always stripped, causing invalid JS output in certain cases. They are now always preserved, even if they would normally qualifiy for vertical whitespace stripping/collapsing when wgResourceLoaderMinifyJSVerticalSpace is enabled. 2011-01-31 12:48:08 +00:00
Platonides
1f74392405 ParseMaster is designed to use big regexes, with many starting characters.
It makes sense to pcre_study() them.
2011-01-30 14:19:55 +00:00
Trevor Parscal
8d36f30802 Resolves bug 26931, where comments were not only not being properly stripped out, but were being transformed into syntax errors. 2011-01-25 19:00:21 +00:00
Trevor Parscal
2fcb86a6c6 Resolves issue in r80656 where escaped quotes within strings are not handled properly, resulting in the string being truncated. 2011-01-24 19:47:57 +00:00
Roan Kattouw
082620d2f4 Fix r80666: botched variable rename 2011-01-21 17:23:24 +00:00
Trevor Parscal
3fe1056d70 Addresses issues raised in an excellent review of r80656. 2011-01-21 00:03:58 +00:00
Trevor Parscal
951103eeb2 Resolved bug 26791 by replacing JSMin with a new library called JavaScriptDistiller, which is an improved version of the minification bits from JavaScriptPacker, an LGPL library. Good news - it's 2x faster than our optimized JSMin anyways, and more configurable to boot. 2011-01-20 21:57:01 +00:00