Merging from master for 2.8.19

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

fixe...

- add `AST_Export` new variants output
- add tests to `test/compress/`
- update `$propdoc` of ...

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

- `side_effects` will clean up inner statements, so checking for an empty function body should s...

fixes #1724
fixes #1725

- move these optimisations out from `Compressor` to `OutputStream`
- fixes behaviour inconsiste...

fixes #1719

Merging from master for 2.8.17

When fixing catch-related issue in #1715, it tries to optimise for duplicate definitions but did...

fixes #1715

fixes #1710

fixes #1709

The following code prints `1`:

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

now guarded under `dead_code`

fixes #1705

fixes #1704

fixes #1698

`statement_to_expression()` drops `typeof` even if it operates on undeclared variables.

Since...

fixes #1690

Unlike normal variables and even function definitions, these cannot be reassigned, even though a...

Process variable definitions with or without assigned values against:
- `arguments`
- named fu...

fixes #1686

Conditions including strict mode would make `delete` return `true` or `false`, and are too compl...

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

fixes #1679

extension of #1675

@qfox Put value constants in a global constant 74c0fb9
@qfox And the other string based valu...

fixes #1676

- correctly determine reachability of (default) branches
- gracefully handle multiple default b...

`AST_Try` is an `AST_Block`, so besides try block we also need to inspect catch and finally bloc...

Take conditional nature of switch branches into account.

fixes #1670

`has_side_effects()` does not take `typeof`'s magical power of not tripping over undeclared vari...

Need to call `extract_declarations_from_unreachable_code()`.

fixes #1663

Turns out the only place in `Compressor` which can generate invalid `AST_For.init` is within `dr...

closes #1647

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

fixes #1649

Enforce `null` as value for empty initialisation blocks.

fixes #1648

fixes #1645
fixes #1646

take side effects of binary boolean operations into account

fixes #1639

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

`compress()` may modify input ASTs

add tests for #1627 & #1640

fixes #1637

- #1634 bars variables with cross-scope references in between to collapse
- but if assigned val...

Merging from master for 2.8.15

mostly done by @kzc

fixes #1631

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

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

- mention performance anomaly in Node 7 and drop from CI
- remove unused npm "scripts"
- mark ...

fixes #1626

Merging from master for 2.8.14

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

The following is wrong:
`a == (b ? a : c)` => `b`
Because:
- `b` may not be boolean
- `a...

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

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

Merging from master for 2.8.13

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

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

fixes #1613

`reduce_vars` enables substitution of variables but did not clone the value's `AST_Node`.

Thi...

fixes #1588

fixes #1605

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

As patched in #1597, `make_node_from_constant()` makes inconsistent and sometimes incorrect call...

Liberal use of `Compressor.transform()` and `AST_Node.optimize()` presents an issue for look-up ...

Merging from master for 2.8.12

- remove obsolete optimisation in `AST_Binary` after #1477
- improve `TreeWalker.has_directive(...

- `AST_DefaultAssign` on `keep_fargs`
- `AST_Expansion on` `keep_fargs`
- `AST_Destructuring` ...

fixes #1592

Self-referenced function has non-fixed values assigned to its parameters.

Let `unused` & `!ke...

Merging from master for 2.8.11

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

Use array of strings ...

As patched on `harmony`, `statement()` is the only user of `embed_tokens()` with a missing error...

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

...