Age | Commit message (Expand) | Author |
2017-04-02 | fix corner case in `switch` (#1765) | Alex Lam S.L |
2017-04-02 | avoid confusion of `NaN` & `Infinity` with `catch` symbol of the same name (#......fixes #1760
fixes #1761 | Alex Lam S.L |
2017-04-02 | fix corner cases in switch and undefined (#1762)...- fix side effects in switch condition for singular blocks
- fix `undefined` confusion with local variable
- gate `OPT(AST_Switch)` with `switches`
fixes #1758
fixes #1759 | Alex Lam S.L |
2017-04-01 | fix switch branch elimination (#1752)...Merge unreachable case body with previous fallthrough case
fixes #1750 | Alex Lam S.L |
2017-04-01 | improve compression of undefined, NaN & Infinitiy (#1748)...- migrate transformation logic from `OutputStream` to `Compressor`
- always turn `undefined` into `void 0` (unless `unsafe`)
- always keep `NaN` except when avoiding local variable redefinition
- introduce `keep_infinity` to suppress `1/0` transform, except when avoiding local variable redefinition
supersedes #1723
fixes #1730 | Alex Lam S.L |
2017-03-31 | combine rules for binary boolean operations (#1744) | Alex Lam S.L |
2017-03-31 | sort options in alphabetical order (#1743)...They started off as functional groups I guess, but given the sheer number of options this is becoming too difficult to read. | Alex Lam S.L |
2017-03-30 | remove paranthesis for `-(x*y)` (#1732) | Alex Lam S.L |
2017-03-30 | optimize try-catch-finally (#1731)...- eliminate empty blocks
- flatten out if try-block does not throw | Alex Lam S.L |
2017-03-29 | speed up IIFE elimination (#1728)...- `side_effects` will clean up inner statements, so checking for an empty function body should suffice
- drop side effects when dropping `return` from statement | Alex Lam S.L |
2017-03-29 | speed up `equivalent_to()` and `AST_Switch` (#1727) | Alex Lam S.L |
2017-03-29 | output optimal representations of NaN & Infinity (#1723)...- move these optimisations out from `Compressor` to `OutputStream`
- fixes behaviour inconsistency when running uglified code from global or module levels due to redefinition | Alex Lam S.L |
2017-03-29 | fix corner case in `unused` (#1718)...When fixing catch-related issue in #1715, it tries to optimise for duplicate definitions but did not take anonymous functions into account.
Remove such optimisation for now and we can cover this as a more general rule later. | Alex Lam S.L |
2017-03-28 | fix `unused` on var of the same name within catch (#1716)...fixes #1715 | Alex Lam S.L |
2017-03-28 | fix `is_number()` on `+=` (#1714)...fixes #1710 | Alex Lam S.L |
2017-03-28 | drop anonymous function name when overshadowed by other declarations (#1712)...fixes #1709 | Alex Lam S.L |
2017-03-28 | fix tail trimming of switch blocks (#1707)...now guarded under `dead_code`
fixes #1705 | 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 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-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 |