| Age | Commit message (Expand) | Author |
|---|
| 2017-03-28 | fix tail trimming of switch blocks (#1707)...now guarded under `dead_code`
fixes #1705 | Alex Lam S.L |
| 2017-03-28 | fix mangle for variable declared within catch block (#1706)...fixes #1704 | Alex Lam S.L |
| 2017-03-27 | `has_side_effects()` should take `AST_Switch.expression` into account (#1699)...fixes #1698 | Alex Lam S.L |
| 2017-03-27 | fix typeof side effects (#1696)...`statement_to_expression()` drops `typeof` even if it operates on undeclared variables.
Since we now have `drop_side_effect_free()`, replace and remove this deprecated functionality. | Alex Lam S.L |
| 2017-03-27 | preserve side effects in switch expression (#1694)...fixes #1690 | Alex Lam S.L |
| 2017-03-27 | fix `cascade` on anonymous function reference (#1693)...Unlike normal variables and even function definitions, these cannot be reassigned, even though assignment expressions would "leak" the assigned value as normal. | Alex Lam S.L |
| 2017-03-27 | handle overlapped variable definitions (#1691)...Process variable definitions with or without assigned values against:
- `arguments`
- named function arguments
- multiple definitions within same scope
Essentially demote variable declarations with no value assignments.
Also fixed invalid use of `AST_VarDef` over `arguments` - should use a member of `AST_SymbolDeclaration` instead. | Alex Lam S.L |
| 2017-03-26 | fix `delete` related issues in `collapse_vars` and `reduce_vars` (#1689) | Alex Lam S.L |
| 2017-03-26 | fix `cascade` on `delete` operator (#1687)...Conditions including strict mode would make `delete` return `true` or `false`, and are too complex to be evaluated by the compressor.
Suppress assignment folding into said operator.
fixes #1685 | Alex Lam S.L |
| 2017-03-26 | fallthrough should not execute case expression (#1683)...- de-duplicate trailing cases only, avoid all potential side-effects
- enable switch statement fuzzing
fixes #1680 | Alex Lam S.L |
| 2017-03-26 | optimize conditional when condition symbol matches consequent (#1684) | kzc |
| 2017-03-26 | suppress switch branch de-duplication upon side effects (#1682)...fixes #1679 | Alex Lam S.L |
| 2017-03-26 | fix side-effects detection on switch statements (#1678)...extension of #1675 | Alex Lam S.L |
| 2017-03-26 | improve switch optimisations (#1677)...- correctly determine reachability of (default) branches
- gracefully handle multiple default branches
- optimise branches with duplicate bodies
fixes #376
fixes #441
fixes #1674 | Alex Lam S.L |
| 2017-03-25 | fix `has_side_effects()` (#1675)...`AST_Try` is an `AST_Block`, so besides try block we also need to inspect catch and finally blocks for possible side effects.
Also extend this functionality to handle `AST_If` and `AST_LabeledStatement` while we are at it.
fixes #1673 | Alex Lam S.L |
| 2017-03-25 | fix `reduce_vars` on `AST_Switch` (#1671)...Take conditional nature of switch branches into account.
fixes #1670 | Alex Lam S.L |
| 2017-03-25 | fix typeof side-effects (#1669)...`has_side_effects()` does not take `typeof`'s magical power of not tripping over undeclared variable into account.
fixes #1668 | Alex Lam S.L |
| 2017-03-25 | fix `dead_code` on `AST_Switch` (#1667)...Need to call `extract_declarations_from_unreachable_code()`.
fixes #1663 | Alex Lam S.L |
| 2017-03-25 | fix invalid `AST_For.init` (#1657)...Turns out the only place in `Compressor` which can generate invalid `AST_For.init` is within `drop_unused()`, so focus the fix-up efforts.
supercedes #1652
fixes #1656 | Alex Lam S.L |
| 2017-03-24 | fix cascade of `evaluate` optimisation (#1654)...Operator has changed, so break out from rest of the rules.
fixes #1649 | Alex Lam S.L |
| 2017-03-24 | fix corner case in `AST_For.init` (#1652)...Enforce `null` as value for empty initialisation blocks.
fixes #1648 | Alex Lam S.L |
| 2017-03-24 | fix assignment extraction from conditional (#1651)...fixes #1645
fixes #1646 | Alex Lam S.L |
| 2017-03-24 | fix assignment substitution in sequences (#1643)...take side effects of binary boolean operations into account
fixes #1639 | Alex Lam S.L |
| 2017-03-24 | improve error marker placement (#1644)...For AST_UnaryPrefix, points to the operator rather than end of expression. | Alex Lam S.L |
| 2017-03-24 | fix regression: CLI options with hyphens like -b ascii-only (#1640)...fixes #1637 | kzc |
| 2017-03-24 | improve collapsible value detection (#1638)...- #1634 bars variables with cross-scope references in between to collapse
- but if assigned value is side-effect-free, no states can be modified, so it is safe to move | Alex Lam S.L |
| 2017-03-23 | account for cross-scope modifications in `collapse_vars` (#1634)...mostly done by @kzc
fixes #1631 | Alex Lam S.L |
| 2017-03-23 | introduce compressor.info() (#1633)...report the following only when `options.warnings = "verbose"`
- unused elements due to inlining
- collpased variables | Alex Lam S.L |
| 2017-03-21 | throw parse error on invalid assignments (#1627)...fixes #1626 | Alex Lam S.L |
| 2017-03-19 | fix commit 88fb83a (#1622)...The following is wrong:
`a == (b ? a : c)` => `b`
Because:
- `b` may not be boolean
- `a` might have side effects
- `a == a` is not always `true` (think `NaN`)
- `a == c` is not always `false` | Alex Lam S.L |
| 2017-03-19 | fix AST_Binary.lift_sequences() (#1621)...Commit eab99a1c fails to account for side effects from compound assignments. | Alex Lam S.L |
| 2017-03-19 | transform String.charAt() to index access (#1620)...Guarded by `unsafe` as `charAt()` can be overridden. | Alex Lam S.L |
| 2017-03-17 | fix chained evaluation (#1610)...`reduce_vars` enables substitution of variables but did not clone the value's `AST_Node`.
This confuses `collapse_vars` and result in invalid AST and subsequent crash.
fixes #1609 | Alex Lam S.L |
| 2017-03-16 | make `collapse_vars` consistent with `toplevel` (#1608)...fixes #1605 | Alex Lam S.L |
| 2017-03-16 | fix `hoist_vars` on `reduce_vars` (#1607)...`hoist_vars` converts variable declarations into plain assignments, which then confuses `reduce_vars`
fixes #1606 | Alex Lam S.L |
| 2017-03-16 | fix stack issues with `AST_Node.evaluate()` (#1603)...As patched in #1597, `make_node_from_constant()` makes inconsistent and sometimes incorrect calls to `optimize()` and `transform()`.
Fix those issues properly by changing the semantics of `evaluate()` and `make_node_from_constant()`, with the side effect that `evaluate()` no longer eagerly converts constant to `AST_Node`. | Alex Lam S.L |
| 2017-03-15 | fix `AST_Node.optimize()` (#1602)...Liberal use of `Compressor.transform()` and `AST_Node.optimize()` presents an issue for look-up operations like `TreeWalker.in_boolean_context()` and `TreeWalker.parent()`.
This is an incremental fix such that `AST_Node.optimize()` would now contain the correct stack information when called correctly. | Alex Lam S.L |
| 2017-03-14 | minor clean-ups (#1600)...- remove obsolete optimisation in `AST_Binary` after #1477
- improve `TreeWalker.has_directive()` readability and resilience against multiple visits | Alex Lam S.L |
| 2017-03-11 | temporary fix for boolean bug (#1597)...fixes #1592 | Alex Lam S.L |
| 2017-03-11 | disallow parameter substitution for named IIFEs (#1596)...Self-referenced function has non-fixed values assigned to its parameters.
Let `unused` & `!keep_fnames` do the scanning, then apply `reduce_vars` only to unnamed functions.
fixes #1595 | Alex Lam S.L |
| 2017-03-10 | fixup for #1585 (#1589)...As patched on `harmony`, `statement()` is the only user of `embed_tokens()` with a missing error branch.
Updated test case and match up with `harmony` to facilitate future merging. | Alex Lam S.L |
| 2017-03-10 | fix catch variable reference in IE8 (#1587)...`AST_Scope.def_variable()` will overwrite `AST_Symbol.thedef`, so save a copy before calling.
fixes #1586 | Alex Lam S.L |
| 2017-03-10 | Correctly raise a parse exception with a missing loop body (#1585) | Michael Mior |
| 2017-03-09 | fix & improve function argument compression (#1584)...- one-use function call => IIFE should take `eval()` & `arguments` into account
- if unused parameter cannot be eliminated, replace it with `0`
fixes #1583 | Alex Lam S.L |
| 2017-03-09 | scan RHS of dropped assignments (#1581)...- similar case as #1578 but against #1450 instead
- fix `this` binding in #1450 as well
closes #1580 | Alex Lam S.L |
| 2017-03-08 | scan assignment value in drop_unused() (#1578)...those were not optimised for `unused` before, which made it necessary for `reduce_vars` to have separate steps for `keep_fnames`
docs update by @kzc
closes #1577 | Alex Lam S.L |
| 2017-03-08 | fix function name eliminiation (#1576)...Function expression can be assigned to a variable and be given a name. Ensure function name is the reduced variable before clearing it out.
fixes #1573
fixes #1575 | Alex Lam S.L |
| 2017-03-08 | plan B for IE8 do-while semi-colon fix (#1572)...- omitting trailing semi-colon in do-while breaks non-browser parser, e.g. uglify-js 1.x
- trailing semi-colon only breaks IE8 if followed by `else` or `while`
- always use braces in do-while body to workaround 2nd case with no size loss in compression
fixes #1568 | Alex Lam S.L |
| 2017-03-08 | fix return from recursive IIFE (#1570)...`side-effects` did not account for IIFEs being able to reference itself thus making its return value potentially significant | Alex Lam S.L |
| 2017-03-07 | fix deep cloning of labels (#1565)...`AST_Label.references` get `.initialize()` to `[]` every time after `.clone()`
So walk down the tree to pick up the cloned `AST_LoopControl` pieces and put it back together. | Alex Lam S.L |
