fix `unsafe` on `evaluate` of `reduce_vars` (#1870)

Determine if variables with non-constant values can escape and be modified.

fixes #1865

kill `opera` (#1869)

enhance `collapse_vars` (#1862)

- extend expression types
  - `a++`
  - `a=x;`
- extend scan range
  - `for(init;;);`
  - `switch(expr){case expr:}`
  - `a = x; a = a || y;`
- terminate upon `debugger;`

closes #1821
fixes #27
fixes #315
fixes #1858

rename variables for better readability (#1863)

update `test/benchmark.js` resources (#1864)

restore report of supported options (#1861)

fixes #1859

improve literal return optimization (#1860)

enforce `toplevel` on other compress options (#1855)

Respect "funcs" and "vars" properly.

fixes #1850

Update ISSUE_TEMPLATE.md (#1846)

fix fuzzer on `this` (#1842)

- forbid redeclaration of `this`
- suppress probability for `this` within nested functions

fix `unused` on for-in statements (#1843)

Only need to avoid `var` within the initialisation block.

fixes #1841

improve parser under "use strict" (#1836)

- `const` without value
- `delete` of expression
- redefining `arguments` or `eval`

extend `test/ufuzz.js`
- optionally generate "use strict"
- improve handling of test cases with syntax errors
- group IIFE generation
- generate bare anonymous functions
- workaround `console.log()` for `new function()`
- generate expressions with `this`

fixes #1810

update README for 3.x (#1840)

fix `AST_For.init` patch-up in `drop_unused()` (#1839)

fixes #1838

fix label-related bugs (#1835)

- deep cloning of `AST_LabeledStatement`
- `L:do{...}while(false)`
- empty statement with label within block

extend `test/ufuzz.js`
- generate labels for blocks & loops
- generate for-in statements
- skip suspicious option search if `minify()` errs

fixes #1833

Fix API reference examples (#1834)

improve `unused` (#1832)

- extract leading value with side-effects out of `var` statement
- reduce scanning of `AST_Definitions` from 3 passes to just once

fix `unused` on labeled for-loop (#1831)

fixes #1830

improve `collapse_vars` on `AST_Var` (#1828)

Perform the same cascaded scanning within `var` statement as we do on array of statements.

fix parser bugs & CLI reporting (#1827)

fixes #1825

extend `cascade` into `a.b` (#1829)

fixes #27

clean up `collapse_vars` (#1826)

- remove overlap in functionality of singular, consecutive reference of constant value
- remove workarounds for previous bugs in `lib/scope.js`
- distribute recursive `collapse_single_use_vars()` calls to their respective `OPT(AST_Node)`
- enable collapsing of variables within a single `AST_Definitions`

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