v2.8.19

fix catch symbol mangling (#1734)

Only need to look up the immediate non-block/catch scope for the same-name special case.

fixes #1733

remove paranthesis for `-(x*y)` (#1732)

optimize try-catch-finally (#1731)

- eliminate empty blocks
- flatten out if try-block does not throw

improve tests from #1726 (#1729)

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

speed up `equivalent_to()` and `AST_Switch` (#1727)

fix missing parentheses around NaN/Infinity shorthands (#1726)

fixes #1724
fixes #1725

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

improve beautified output of switch blocks (#1721)

v2.8.18

remove UGLIFY_DEBUG (#1720)

fixes #1719

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.

v2.8.17

fix `unused` on var of the same name within catch (#1716)

fixes #1715

fix `is_number()` on `+=` (#1714)

fixes #1710

drop anonymous function name when overshadowed by other declarations (#1712)

fixes #1709

handle var within catch of the same name (#1711)

The following code prints `1`:

var a = 1;
!function(){
  a = 4;
  try{
    throw 2;
  } catch (a) {
    var a = 3;
  }
}();
console.log(a);

fixes #1708

fix tail trimming of switch blocks (#1707)

now guarded under `dead_code`

fixes #1705

fix mangle for variable declared within catch block (#1706)

fixes #1704

ufuzz: workaround for Function.toString() v2 (#1700)

`has_side_effects()` should take `AST_Switch.expression` into account (#1699)

fixes #1698

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.

preserve side effects in switch expression (#1694)

fixes #1690

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.

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.

fix `delete` related issues in `collapse_vars` and `reduce_vars` (#1689)

ufuzz: workaround function name and toString() (#1688)

fixes #1686

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

fallthrough should not execute case expression (#1683)

- de-duplicate trailing cases only, avoid all potential side-effects
- enable switch statement fuzzing

fixes #1680

optimize conditional when condition symbol matches consequent (#1684)

suppress switch branch de-duplication upon side effects (#1682)

fixes #1679

fix side-effects detection on switch statements (#1678)

extension of #1675

Improve fuzzer. :) (#1665)

@qfox Put value constants in a global constant 74c0fb9
 @qfox And the other string based values as well a5033c5
 @qfox Be more strict about parameters, allow max to be optional 9c7ce70
 @qfox Support a `V` (capital) flag to only log out at intervals 2d822c7
 @qfox Fewer magic variables a6a9a7c
 @qfox Fix decrement such that a function is created when n=1 7e4b017
 @qfox Add more values 64e596e
 @qfox Make `b` appear more often d33191a
 @qfox Add functions that contain (only..) functions 29a86e3
 @qfox Allow the block statement to contain multiple statements 7570484
 @qfox Make the interval count a constant d587ad8
 @qfox Enable mangling, disable post-processing  â€¦ 4dc8d35
 @qfox Add more simple value that may trigger syntactic errors 8496d58
 @qfox Add `else` to some `if` statements a4aed65
 @qfox Move iife to expr generator, fix missing recursion arg e453159
 @qfox Improve output on error where it wasnt printing the last code properly 4565a1a
 @qfox Add switch statement to generator ceafa76
 @qfox Add var statement, support optional comma for expr generator b83921b
 @qfox Expression generator should use a simple value instead of `0` as recu…  â€¦ 9d1a5c7
 @qfox const -> var to keep things es5... 0143099
 @qfox Add more simple values that may trigger edge cases 5e124f1
 @qfox Add central name generator, take special care for global functions aeb7682
 @qfox Add some `return` and function declaration cases to statement generator 6c9c3cc
 @qfox Exclude switches from generator for now 91124b2

Put value constants in a global constant

And the other string based values as well

Be more strict about parameters, allow max to be optional

Support a `V` (capital) flag to only log out at intervals

Fewer magic variables

Fix decrement such that a function is created when n=1

Add more values

Make `b` appear more often

Add functions that contain (only..) functions

Allow the block statement to contain multiple statements

Make the interval count a constant

Enable mangling, disable post-processing

Mangling is kind of the whole point...

Similarly, to beautify the minified code afterwards may supress bugs so it's probably best not to beautify the code prematurely. And there's no point anyways since you won't see it most of the time and only care about the main input anyways.

Add more simple value that may trigger syntactic errors

Add `else` to some `if` statements

Move iife to expr generator, fix missing recursion arg

Improve output on error where it wasnt printing the last code properly

Add switch statement to generator

Add var statement, support optional comma for expr generator

Expression generator should use a simple value instead of `0` as recursion default

const -> var to keep things es5...

Add more simple values that may trigger edge cases

Add central name generator, take special care for global functions

Add some `return` and function declaration cases to statement generator

Exclude switches from generator for now

Enable switch generation because #1667 was merged

Add typeof generator

Add some elision tests

Add a new edge case that returns an object explicitly

Add all binary ops to try and cover more paths

Forgot four binops and added `Math` to var name pool

Harden the incremental pre/postfix tests

Improve switch generator, allow `default` to appear at any clause index

Add try/catch/finally generation

Prevent function statements being generated

Add edge case with decremental op and a group

Disable switch generation until #1679 and #1680 are solved

Only allow `default` clause as last clause for now

Tentatively enable `throw`, `break` and `continue` statements when in valid contexts

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

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

fix `reduce_vars` on `AST_Switch` (#1671)

Take conditional nature of switch branches into account.

fixes #1670

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

fix `dead_code` on `AST_Switch` (#1667)

Need to call `extract_declarations_from_unreachable_code()`.

fixes #1663

v2.8.16

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

introduce ufuzz.js (#1655)

closes #1647

Update ISSUE_TEMPLATE.md

fix cascade of `evaluate` optimisation (#1654)

Operator has changed, so break out from rest of the rules.

fixes #1649

fix corner case in `AST_For.init` (#1652)

Enforce `null` as value for empty initialisation blocks.

fixes #1648

fix assignment extraction from conditional (#1651)

fixes #1645
fixes #1646

fix assignment substitution in sequences (#1643)

take side effects of binary boolean operations into account

fixes #1639

improve error marker placement (#1644)

For AST_UnaryPrefix, points to the operator rather than end of expression.

fix expect_stdout (#1642)

`compress()` may modify input ASTs

add tests for #1627 & #1640

fix regression: CLI options with hyphens like -b ascii-only (#1640)

fixes #1637

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

v2.8.15

account for cross-scope modifications in `collapse_vars` (#1634)

mostly done by @kzc

fixes #1631

introduce compressor.info() (#1633)

report the following only when `options.warnings = "verbose"`
- unused elements due to inlining
- collpased variables

fix a bug in simple_glob (#1632)

- "?" should not match "/"
- other minor clean-ups

metadata cleanup (#1630)

- mention performance anomaly in Node 7 and drop from CI
- remove unused npm "scripts"
- mark browserify dependency as optional
- stop `test/mozilla-ast.js` from spamming console output in later versions of Node.js

throw parse error on invalid assignments (#1627)

fixes #1626

v2.8.14

make `expect_stdout` work on Node.js 0.12 (#1623)

That particular version of Node.js has messed up error messages, so provide a version-specific workaround.

Also fixed an formatting issue which would cause `expect_stdout` to fail if error message contains excerpts of input.

Apply `expect_stdout` to more applicable tests.

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`

fix AST_Binary.lift_sequences() (#1621)

Commit eab99a1c fails to account for side effects from compound assignments.

transform String.charAt() to index access (#1620)

Guarded by `unsafe` as `charAt()` can be overridden.

handle runtime errors in `expect_stdout` (#1618)

allow test to pass if both `input` and `expect` throws the same kind of error

fix top-level directives in compress tests (#1615)

`input` and `expect` are parsed as `AST_BlockStatement` which does not support `AST_Directive` by default.

Emulate that by transforming preceding `AST_SimpleStatement`s of `AST_String` into `AST_Directive`.

Add `--in-source-map inline` documentation (#1611)

v2.8.13

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

extend `test/run-tests.js` to optionally execute uglified output (#1604)

fixes #1588

make `collapse_vars` consistent with `toplevel` (#1608)

fixes #1605

fix `hoist_vars` on `reduce_vars` (#1607)

`hoist_vars` converts variable declarations into plain assignments, which then confuses `reduce_vars`

fixes #1606

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`.

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.

minor clean-ups (#1600)

- remove obsolete optimisation in `AST_Binary` after #1477
- improve `TreeWalker.has_directive()` readability and resilience against multiple visits

v2.8.12

temporary fix for boolean bug (#1597)

fixes #1592

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

support multi-line string in tests (#1590)

`expect_exact` sometimes have multiple lines and `\n` are hard to read.

Use array of strings to emulate line breaks and improve readability.

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.

v2.8.11

fix catch variable reference in IE8 (#1587)

`AST_Scope.def_variable()` will overwrite `AST_Symbol.thedef`, so save a copy before calling.

fixes #1586

Correctly raise a parse exception with a missing loop body (#1585)

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

v2.8.10

scan RHS of dropped assignments (#1581)

- similar case as #1578 but against #1450 instead
- fix `this` binding in #1450 as well

closes #1580

remove checkboxes from Issues template

explain how to make a proper bug report (#1579)

fixes #1574

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

v2.8.9

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

only run benchmark & jetstream on CI (#1571)

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

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

v2.8.8

include benchmark.js in test suite (#1564)

- report file sizes and overall run time
- exit with non-zero code upon error

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.

transform function calls to IIFEs (#1560)

- expose function body to call sites for potential optimisations
- suppress substitution of variable used within `AST_Defun`

avoid substitution of global variables (#1557)

- unless `toplevel` is enabled
- global `const` works as before

collapse_vars: do not replace a constant in loop condition or init (#1562)

suppress semicolons after do/while (#1556)

- unless both `beautify` & `screw-ie8` are enabled
- deprecate workaround for if-do-while-else

fixes #186

v2.8.7