Don't compress (0, eval)() to eval()

Preserve ThisBinding in conditionals & collapse_vars

Fixes #973

Merge pull request #948 from kzc/collapse_vars_doc

collapse_vars: document the compress option in README

Revert "using the original sourcemap as the base"

This reverts commit ad18689d926d25c7a25b95c630c2ad05b7b5f5b5.

Reason for revert: introduce issue #882

Currently, generated sourcemap contains copy of all existing mappings and adds new mappings from uglified code to original one.
However, previous mapping are no longer valid and shouldn't be added.

Allow cli options to be specified in separate definitions

Fix for #963. This allows stuff like `--define a=1 --define b=1` besides only `--define a=1,b=1`

Create and map `bare-returns` into new `parse` property name

Fixes #951 missing export for SymbolDef

Merge pull request #949 from kzc/collapse_vars_conditions

collapse_vars: fix if/else and ternary operator side effects

collapse_vars: avoid replacement across AST_Case nodes to be on safe side even though no issues seen.

collapse_vars: fix if/else and ternary operator side effects

collapse_vars: small change to README

collapse_vars: document option in README

collapse_vars: fix bug in repeated var defs of same name

Add mangleProperties documentation to README

Add additional documentation to mangleProperties.

fix bug in collapse_vars for right side of "||" and "&&"

Collapse single use var definitions

Fix #721

Add start/end in the `arguments` definition

(keeps my https://github.com/mishoo/jsinfo.el working)

Added a mangle properties option

Merge pull request #928 from STRML/constPragma

Mark vars with /** @const */ pragma as consts so they can be eliminated.

Tighten up @const regex.

Simplify by skipping extra tree walk.

Use TreeWalker for more accurate @const results and update tests

Update README for /** @const */

Mark vars with /** @const */ pragma as consts so they can be eliminated.

Fixes older browser support for consts and allows more flexibility
in dead code removal.

Allow operator names as getters/setters

Fixes #919

Fix provided by @kzc

Add keywords to package.json

Should hopefully bump up on the results of the npm site when searching `uglify`

Add some tests for comment-filters through api

Also never bother comment options to filter comment5/shebang comments
as they have their custom filter.

Merge pull request #918 from avdg/fix-arguments-handling

Never mangle arguments and keep them in their scope

Make arguments test slightly more strict

Add scope test for arguments

Never mangle arguments and keep them in their scope

Fixes #892

Helped-by: kzc

Merge pull request #905 from avdg/unit-tests

Add unit tests

Add test case for line continuation

Fix hoisting the var in ForIn

Close #913

Add mocha tests

Fix ch that could contain other newline characters

Do not allow newlines in string literals

Merge pull request #874 from kzc/fix-conditionals

#873 Fix `conditionals` optimizations with default compress options

Merge pull request #896 from avdg/do-while-semicolon

Semicolon after do...while statement is optional

Disable loop optimization for parse-only tests

Add tests

Semicolon after do...while statement is optional

Merge pull request #879 from ReadmeCritic/master

Update README URLs based on HTTP redirects

Update README URLs based on HTTP redirects

#873 Fix `conditionals` optimizations with default compress options

v2.6.1

Fix endless loop

Close #866

Limit max iterations for tighten_body

Ref #866

Merge pull request #864 from plievone/patch-1

Fix docs for keep_fargs

Fix docs for keep_fargs

Compression options `keep_fargs` and `unsafe` were decoupled in v.2.5.0 (commit 5fd1245), so document actual keep_fargs default.

v2.6.0

Fix output for "use asm" code from SpiderMonkey AST

(will only work properly if the SM tree contains "raw" properties for
Literal number nodes)

Build label def/refs info when figuring out scope

Fix #862

Replace util.error with console.log

Rework has_directive

It's now available during tree walking, i.e. walker.has_directive("use
asm"), rather than as part of the scope.  It's thus no longer necessary
to call `figure_out_scope` before codegen.  Added special bits in the
code generator to overcome the fact that it doesn't inherit from
TreeWalker.

Fix #861

Merge pull request #854 from kzc/moz-regexp-2

Have mozilla AST RegExpLiteral parser use regex.pattern and regex.flags

Fix parsing invalid input

i.e. `x = 1.xe` â€” because parseFloat("1.xe") returns 1, this parsed as
`x = 1`.

Ref #857

Prevent ReDoS by not using a regexp to verify floating point numbers

`parseFloat` will return `NaN` for invalid numbers anyway, which is the check used to throw the parse error.

Fixes #857

simplify mozilla AST RegExpLiteral token parse and handle corner cases of regex.pattern better

Have mozilla AST RegExpLiteral parser use regex.pattern and regex.flags rather than non-standard `raw` property.

rename To_Moz_Literal to To_Moz_RegExp

Fixed RegExp literal in mozilla AST generation/output and added a --dump-spidermonkey-ast flag

`return undefined` optimization no longer uses `return_void_0` option

more tests for `return undefined` optimization

optimize `return undefined` and `return void 0`

Allow specification beautify options in tests

Caught an error in #847 as well - `output` wasn't passed anywhere which led to an exception. `options` was available though.

Stop building for io.js

fixes #845: \v escaping should be restricted to "screw_ie8" mode

Fix #836

Make_string was missing \v and wasnt reversing vertical tabs even though read_escaped_char coverts them

add `--pure-funcs` option

it has the same effect as specifying `pure_funcs` in `--compressor`
option, however it's much easier to use

instead of:

    --compressor 'pure_func=["Math.floor","debug","console.logTime"]'

it's now possible:

    --compressor --pure-funcs Math.floor debug console.logTime

fixes #684

upgrade yargs 3.5.4 -> 3.10.0

we need a version with better support for 'array' params
see: https://github.com/bcoe/yargs/pull/164

Merge pull request #829 from kzc/html_comment_ops

Fix other operator output producing <!-- or -->

Fix other operator output related to <!-- or -->

Fix tests

Only encode <!-- and --> in strings when inline_script

Output `-- >` instead of `-->` in expressions. Escape `<!--` and `-->` within string literals.

v2.5.0

have minify() call figure_out_scope() if needed to produce well formed "use asm" code

get rid of SCOPE_IS_NEEDED as it was always true

Fix handling of "use asm" when no command line flags are passed to uglifyjs. SCOPE_IS_NEEDED is unconditionally true now. Refactored floating point literal parsing to be more in keeping with the AST class design.

Pin dependencies with npm shrinkwrap

* Use `npm run shrinkwrap` to create a shrinkwrap file with all dependencies pinned
* Update dependency `source-map` to latest (Closes #738)

add asm.js test

Add proper support for "use asm"; blocks. Disable -c optimization within "use asm"; sections and preserve floating point literals in their original form. Non-asm.js sections are optimized as before. Asm.js sections can still be mangled and minified of whitespace. No special command line flags are required.

Implement shebang support

Merge pull request #808 from avdg/travis

Add node 4.x in Travis

Add node 4.x in Travis

Disable node 0.8 in Travis

Fix mozilla-ast after module loading changes

Need to explicitly qualify stuff now, since it's not evaluated in some
global scope.

Ref #636

No longer use `vm` to load code.

Improves performance 2x on node > 0.10.

Ref #636

Fix wrap_commonjs to include code first

(code could have directives, i.e. "use strict")

use a valid SPDX license identifier

Control keeping function arguments with a single option

Merge pull request #786 from istr/anonymous-source-map

Allow for anonymous map generation using string type check

allow for anonymous map generation using string type check

Fix bad parsing of `new new x()()` constructs

Fixes #739

Don't change sequences that influence lexical binding in calls

Fixes #782

Merge pull request #767 from vjeux/208

[Fix] --define replaces SymbolRefs in LHS of assignments

[Fix] --define replaces SymbolRefs in LHS of assignments

See #208 for context

Merge pull request #736 from AlbertoGP/master

fromString option, use index from argument array for filename instead of "?"