support safe reassignments in `reduce_vars` (#1823)

`var a=1;a=2;x(a)` => `x(2)`

fix pre-existing issues
- reference counting on assignment
- walking of anonymous functions
- chained assignment

fix `reduce_vars` on conditionals (#1822)

fix `reduce_vars` on boolean binary expressions (#1819)

Side effects of `&&` and `||` have not mattered until #1814, which takes assignment expressions into account.

compress duplicated variable definitions (#1817)

These are surprisingly common, as people reuse the same variable name within loops or switch branches.

fix `reduce_vars` within try-block (#1818)

Possible partial execution due to exceptions.

enhance `reduce_vars` (#1814)

- allow immediate assignment after declaration of variable
- relax modification rule for immutable value
- fix order of visit for TreeWalker
- remove extraneous code

fix variable substitution (#1816)

- let `collapse_vars` take care of value containing any symbols
- improve overhead accounting

update README (#1813)

- mention major version bump
- remove reference to internal API

unify CLI & API under `minify()` (#1811)

- rename `screw_ie8` to `ie8`
- rename `mangle.except` to `mangle.reserved`
- rename `mangle.properties.ignore_quoted` to `mangle.properties.keep_quoted`
- compact `sourceMap` options
- more stringent verification on input `options`
- toplevel shorthands
  - `ie8`
  - `keep_fnames`
  - `toplevel`
  - `warnings`
- support arrays and unquoted string values on CLI
- drop `fromString` from `minify()`
  - `minify()` no longer handles any `fs` operations
- unify order of operations for `mangle_properties()` on CLI & API
  - `bin/uglifyjs` used to `mangle_properties()` before even `Compressor`
  - `minify()` used to `mangle_properties()` after `Compressor` but before `mangle_names()`
  - both will now do `Compressor`, `mangle_names()` then `mangle_properties()`
- `options.parse` / `--parse` for parser options beyond `bare_returns`
- add `mangle.properties.builtins` to disable built-in reserved list
  - disable with `--mangle-props builtins` on CLI
- `warnings` now off by default
- add `--warn` and `--verbose` on CLI
- drop `--enclose`
- drop `--export-all`
- drop `--reserved-file`
  - use `--mangle reserved` instead
- drop `--reserve-domprops`
  - enabled by default, disable with `--mangle-props domprops`
- drop `--prefix`
  - use `--source-map base` instead
- drop `--lint`
- remove `bin/extract-props.js`
- limit exposure of internal APIs
- update documentations

closes #96
closes #102
closes #136
closes #166
closes #243
closes #254
closes #261
closes #311
closes #700
closes #748
closes #912
closes #1072
closes #1366
fixes #101
fixes #123
fixes #124
fixes #263
fixes #379
fixes #419
fixes #423
fixes #461
fixes #465
fixes #576
fixes #737
fixes #772
fixes #958
fixes #1036
fixes #1142
fixes #1175
fixes #1220
fixes #1223
fixes #1280
fixes #1359
fixes #1368

drop `angular` (#1812)

Remove support for `@ngInject` as there are proper alternatives anyway.

convert `AST_Seq` from binary tree to array (#1460)

- rename `AST_Seq` to `AST_Sequence`
- raise default sequences_limit from 200 to 800

v2.8.22

fix LHS cases for NaN & friends (#1804)

`Infinity = beyond` should not become `1/0 = beyond`

enhance `test/ufuzz.js` (#1803)

- `-E` to report test cases with runtime errors
- favor returning expressions rather than empty return
- emit a newline upon fuzzer completion to not erase the iteration count

closes #1800

fix a couple of bugs in `global_defs` (#1802)

- `optimize()` substituted expression
- compute nested property string correctly

fixes #1801

Miscellaneous
- reset optimisation flags on all node types

fix `delete` corner cases (#1799)

- assignment
- boolean
- conditional
- sequence

extend ufuzz generator (#1783)

- property access
- property assignment
- allow bare expression within try-block
- normalise `Error` in `console.log()`
- generate more unary expressions
- add parenthesis to enforce precedence
- adjust variable reuse/creation
- add parameters to function declaration & expression
- add return expression
- add trivial arguments to function call

fix `pure_getters` for chained property access (#1798)

fix corner cases with `delete` (#1796)

`delete Infinity` returns `false` where as `delete (1/0)` returns `true`

introduce "strict" to `pure_getters` (#1795)

fix `reduce_vars` on catch variable (#1794)

Improved catch handling in `figure_out_scope()` means special case treatment of IE8 is no longer valid in `reset_opt_flags()`.

Also fixed recursive assignment in variable definition.

fix incorrect context in variable substitution (#1791)

`AST_Node.optimize()` is context-aware, so don't cache its results to be used elsewhere.

Also fixed a few cases of AST corruption and beef up safety of `pure_getters`.

enable `inline_script` by default (#1793)

improve `pure_getters` (#1786)

- property access to `null` & `undefined` always has side effects
- utilise `reduce_vars` to determine safe property access
- may-be cases treated as side effects unless `unsafe`

implement delayed resolution for `reduce_vars` (#1788)

Although it would be nice to enforce `AST_Node` cloning during transformation, that ship has sailed a long time ago.

We now get the assigned value when resolving `AST_SymbolRef` instead of `reset_opt_flags()`, which has the added advantage of improved compressor efficiency.

fixes #1787

optimise `do{...}while(false)` (#1785)

- better heuristics to avoid issues like #1532
- fix `TreeWalker.loopcontrol_target()`
  - `continue` cannot refer to `switch` blocks

remove `--mangle-props` from fuzzing (#1777)

The inherently unsafe nature makes this feature unsuitable to be tested this way.

fixes #1774

exclude mangling of special property names (#1779)

- `null`
- `true`
- `false`
- numeric literals

fix `mangleProperties` on identifiers (#1776)

- fix handling of "-Infinity"
- add test case for "-0"

reverts #1481

workaround Node.js bugs (#1775)

Wrap test code in IIFE before passing to `vm`

fixes #1768
fixes #1771

fix mangleProperties of `undefined` & `Infinity` (#1772)

`NaN` already works by the happy accident of `Number.NaN`

fixes #1770

extend `test/ufuzz.js` (#1769)

New expressions:
- property access
- array literal
- object literal

Miscellaneous:
- reduce execution timeout
- test `toplevel` and `mangleProperties`

v2.8.21

fix corner case in `switch` (#1765)

avoid confusion of `NaN` & `Infinity` with `catch` symbol of the same name (#1763)

fixes #1760
fixes #1761

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

speed up fuzzer code generation (#1757)

- only output one top-level function or statement block
- reduce `rng()` granularity from 2^32 to 65536
- fix overflow in `rng()`
- track `canThrow` during `typeof` creation

minor tweaks to `test/ufuzz.js` (#1756)

- count iterations from `1` instead of `0`
- remove `unsafe` from default set of `minify()` tests
- improve usability of help

upgrade fuzzer (#1754)

- configurable set of `minify()` options
- test and report suspects upon failure
- continue after failure if infinite iterations is specified

fuzz regexp literals, more constant numbers, typeof expression (#1755)

fix switch branch elimination (#1752)

Merge unreachable case body with previous fallthrough case

fixes #1750

minor tweaks to fuzzer (#1751)

- remove `let` as variable name
- employ `crypto.randomBytes()`

implement `test/sandbox.js` (#1749)

- `test/run-tests.js` and `test/ufuzz.js` now shares the same `run_code()` and `same_stdout()`
- re-enable fuzzer to generate top-level `NaN`, `Infinity` & `undefined`
- attempt to show beautified output only when `run_code()` output is preserved

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

combine rules for binary boolean operations (#1744)

Massive extension of the fuzzer (#1697)

Fix bug where a `throw` was generated without expression

Reenable try/catch/finally and fix them up

Skip serialization errors

Allow function decl in other funcs but not in blocks etc

Rename function to be more appropriate

Fix global functions not getting certain names

Make the canaries more likely to appear as expressions

Add a silly rounding edge case

Add a new canary, `c`, which should only ever be incremented

Refactoring

Fix (another) iife not actually being invoked

When a statement hits recursion max return an expression instead of `;`

When a expression hits recursion max also inc `c`

Generate global code as well as function code

Also fixes some argument juggling related bugs.
No longer reduces the recursion max when generating sub functions.
Generates a function arg.

Add used names to var name pool while in that scope

This is a little wonky, possibly a hack, but since it's synchronous code I think it's alright to do this. The alternative is to slice the varnames array and juggle them through almost all the generator functions and there are various reasons why this patch is a better alternative.

Minify generated code, not beautified code. Prevents beautifier bias.

Prevent unnecessary duplication

Remove serialization protection because I think it got handled elsewhere

Abstract toplevel code generation

Add example line of running test case

Add poor man options parser, and some options

Reindent to 4 spaces

Lower chance of `default` generation

Comment example of testing a case and output improvement

Enable `default` clause appearing at any clause index

Removing some training wheels; dont add parens where we dont absolutely need them

Support `-s1` and `-s2` to force specific statements being generated at that recursion level

Add round number to output when failing. For stats and fun and profit.

Solidify statement depth counting. The argument juggling is real.

Renamed option to something long. -scf was ugly and probably confusing.

Fix missing arguments causing `canThrow` to be truthy, generating crashing code

Generate more binary nested expressions

Add black and white list cli options for statement generation

Allows you to explicitly require or forbid certain statements from/to being made.

```
node test/ufuzz.js --without-stmt switch,try -t 5 -r 5 -V
```

```
node test/ufuzz.js --only-stmt ifelse,expr -t 5 -r 5 -V
```

Similar granularity for expression may be added later.

There can be no comma between names; it just does a split on that arg.

Trim down the binary expression generator

Prevent scoping issues in nodejs by preventing certain names in global space

Oh this list was incomplete?

Allow bin-expr to generate assignments too. More vigilant with storing and reusing vars.

Add more global builtin names

Update wrapper code

Also patch Function valueOf

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.

v2.8.20

fix missing preamble when shebang is absent (#1742)

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