fix `evaluate` on object getter & setter (#1515)

v2.8.1

temporarily disables `reduce_vars` (#1517)

... as we investigate #1516

warn & drop `#__PURE__` iff IIFE is dropped (#1511)

- consolidate `side-effects` optimisations
- improve string `+` optimisation
- enhance literal & `conditionals` optimisations

v2.8.0

improve error messages (#1506)

- better inheritance of `Error` sub-classes
- mark parse error against source in CLI

closes #235
closes #348
closes #524
closes #1356
closes #1405

add harmony branch details in README (#1507)

faster tree transversal (#1462)

- convert `[].forEach()` to for-loops

consolidate `evaluate` & `reduce_vars` (#1505)

- improve marking efficiency
- apply smarter `const` replacement to `var`

fixes #1501

update docs for `pure_funcs` & `drop_console` (#1503)

closes #1362
closes #1399

allow --in-source-map inline (#1490)

- limited to one input file (or `stdin`)
- only works with built-in parser

fixes #520

Avoid using exports when undefined (#1471)

Makes direct usage within web browser easier, even if officially unsupported.

Merge pull request #1485 from alexlamsl/merge-2.8.0

2.8.0 staging

enable `collapse_vars` & `reduce_vars` by default
- fix corner cases in `const` optimisation
- deprecate `/*@const*/`

fixes #1497
closes #1498

Support marking a call as pure

A function call or IIFE with an immediately preceding comment
containing `@__PURE__` or `#__PURE__` is deemed to be a
side-effect-free pure function call and can potentially be
dropped.

Depends on `side_effects` option.

`[#@]__PURE__` hint will be removed from comment when pure
call is dropped.

fixes #1261
closes #1448

Fix: AST_Accessor missing start / end tokens

fixes #1492
closes #1493

fix `pure_funcs` & improve `side_effects`
- only drops side-effect-free arguments
- drop side-effect-free parts with discarded value from `AST_Seq` & `AST_SimpleStatement`

closes #1494

clean up `max_line_len`
- never exceed specified limit
- otherwise warning is shown
- enabled only for final output

closes #1496

introduce `unsafe_proto`
- `Array.prototype.slice` => `[].slice`

closes #1491

remove extraneous spaces between ++/+/--/-

fixes #1377
closes #1488

Add .gitattributes to checkout lf eol style

closes #1487

improve `--beautify bracketize`
reduce whitespaces from if-else statements

fixes #1482
closes #1483

add benchmark & JetStream tests
- `test/benchmark.js` measures performance
- `test/jetstream.js` verifies correctness
- configurable mangle/compress/output options

closes #1479

enhance `global_defs`
- support arrays, objects & AST_Node
- support `"a.b":1` on both cli & API
- emit warning if variable is modified
- override top-level variables

fixes #1416
closes #1198
closes #1469

evaluate AST_SymbolRef as parameter
fix invalid boolean conversion now exposed in `make_node_from_constant()`

closes #1477

improve reduce_vars and fix a bug
- update modified flag between compress() passes
- support IIFE arguments
- fix corner case with multiple definitions

closes #1473

improve keep_fargs & keep_fnames
- utilise in_use_ids instead of unreferenced()
- drop_unused now up-to-date for subsequent passes

closes #1476

fix duplicated test names

previously test cases with the same name would be skipped except for the last one

`test/run-test.js` will now report duplicated names as errors

closes #1461

smarter const replacement taking name length into account

closes #1459

clean up `negate_iife`
- remove extra tree scanning phase for `negate_iife`
- `negate_iife` now only deals with the narrowest form, i.e. IIFE sitting directly under `AST_SimpleStatement`
- `booleans`, `conditionals` etc. will now take care the rest via more accurate accounting
- `a(); void b();` => `a(); b();`

fixes #1288
closes #1451

improve string concatenation
shuffle associative operations to minimise parentheses and aid other uglification efforts

closes #1454

`-c sequences=N` suboptimal at N expression cutoff

N = 2:
  a;
  b;
  c;
  d;
was:
  a, b;
  c;
  d;
now:
  a, b;
  c, d;

fixes #1455
closes #1457

tweak do-while loops
- `do{...}while(false)` => `{...}`
- clean up `AST_While` logic

closes #1452

fixes & improvements to [].join()

fixes
- [a].join() => "" + a
- ["a", , "b"].join() => "a,,b"
- ["a", null, "b"].join() => "a,,b"
- ["a", undefined, "b"].join() => "a,,b"

improvements
- ["a", "b"].join(null) => "anullb"
- ["a", "b"].join(undefined) => "a,b"
- [a + "b", c].join("") => a + "b" + c

closes #1453

drop unused: toplevel, assign-only
- assign statement does not count towards variable usage by default
- only works with assignments on the same scope level as declaration
- can be disabled with `unused` set to "keep_assign"
- `toplevel` to drop unused top-level variables and/or functions
- `top_retain` to whitelist top-level exceptions

closes #1450

fix stray else in compress with conditionals=false

closes #1449

enable typeof "undefined" for general use
move out of unsafe, guard corner case with screw_id8 instead

closes #1446

fix corner case in keep_fnames
happens when inner function:
- just below top level
- not referenced
- `unused` is disabled

closes #1445

remove unused AST_Scope.nesting & AST_SymbolRef.frame
they are computed but never used

closes #1444

fix crash in unsafe replacement of undefined
remove extraneous call to AST_SymbolRef.reference()

closes #1443

Improve optimizing `function() { if(c){return foo} bar();}`

closes #1437

Merge pull request #1481 from anatdagan/propsmangle_only_identifiers

verify that property names after mangle are legal

in mangle_names there is a check that the variable name is legal and that it is not a reserved word. This should apply to propsmangle as well.

update test (#1441)

improved reduce_vars & binary operands produce more optimal results

fix mangling collision with keep_fnames (#1431)

* fix mangling collision with keep_fnames
fixes #1423

* pass mangle options to figure_out_scope()
bring command-line in line with minify()

optimise binary operands with evaluate() (#1427)

- remove call to evaluate() in is_constant() and let nested optimize() does its job instead
- reject RegExp in is_constant() and remove special case logic under collapse_vars
- operands to conditionals optimisation are now always evaluate()-ed
- throw error in constant_value() instead of returning undefined to catch possible bugs, similar to make_node_from_constant()
- optimise binary boolean operators under `evaluate` instead of `conditionals`

augment evaluate to extract within objects (#1425)

- gated by `unsafe`
- replaces previous optimisation specific to String.length
- "123"[0] => 1
- [1, 2, 3][0] => 1
- [1, 2, 3].length => 3
- does not apply to objects with overridden prototype functions

add missing LHS cases which global_defs should avoid

Have minify() and tests use figure_out_scope() as uglifyjs CLI does

Clarify docs, help and tests for --support-ie8 and screw_ie8=false

Add preventive test involving non-ascii function identifiers

Add note about name mangling when using --mangle-props=unquoted (#1314)

remove npm-shrinkwrap.json to work around npm@4.0.2 bug (#1384)

v2.7.5

Generate source map data from normalized files

If using `inSourceMap` this fix will ensure the copying of `sourcesContent` is based on potentially normalized `sources` values (https://github.com/mozilla/source-map/blob/master/lib/source-map-consumer.js#L304-L309).

For example `normalize` (https://github.com/mozilla/source-map/blob/master/lib/util.js#L80-L123) will rewrite `./dist/mySource.js` to `dist/mySource.js` in the target `_sources` of the `SourceMapConsumer`. As a result `orig_map.sourceContentFor(source, true);` would return `null` since the orginal `source` was no longer available in the consumer. By using the keys generating from the `SourceMapConsumer.constructor` consistency is ensured.

docs: add doc for option.outFileName

feat: add option.outFileName for JS API, if absense, sourceMap.file field will deduced

Add --mangle-props-debug and fix --mangle-props=unquoted collision

Patch by @AshleyScirra

Based on: PR #1316

Renamed the CLI debug option to --mangle-props-debug

Fixes: #1321 name collision in --mangle-props=unquoted

Do not overwrite options.comments + cleanup

Pass mangle options to figure_out_scope before mangling in tests

v2.7.4

Don't filter shebangs when using the 'some' comment filter

Also clarify documentation a bit more about using regexp as filter

fix uses_arguments handling (broken since 6605d1578351)

Using the symbol declaration tracking of UglifyJS doesn't make sense here
anyway, `arguments` always comes from something in the current scope.

fixes #1299

Add an option for writing inline source map

Remove console.log and add extra test case

Add an option to wrap IIFEs in parenthesis

For #1307.

Fix (typeof side_effect()) in boolean context

Fixes #1289 with suggestion by @rvanvelzen

Optimize unmodified variables

Make all comment options in cli available in js api

Also removing more code within "loop" while at it.

Merge pull request #1277 from kzc/fix-string-plus-opt

Account for side effects in `string + expr` optimization

Account for side effects in `string + expr` optimization

implement optimization: (x = 2 * x) ---> (x *= 2)

v2.7.3

Fix negate_iife transform to return a correct tree for nested IIFEs

Fix for #1256, partially reverts d854523783b4

v2.7.2

Fix negate_iife regression #1254

v2.7.1

Add missing `{` in README

Also fix a trivial style mistake.

Fix --mangle-props and --mangle-props=unquoted

Fixes: #1247

Fix --mangle-props and --name-cache inconsistency.
AST_Dot and AST_Sub properties are now mangled by --mangle-props
without regard to being used in an assignment statement.

Note: if --mangle-props is used then *all* javascript files used must
be uglified with the same mangle options.

Fix the ignore_quoted=true mangle option, also known as
`--mangle-props=unquoted`.  If a given property is quoted anywhere
it will not be mangled in any quoted or non-quoted context.

Additional object literal property tests

Add input file glob support to minify()

Add simple file globbing to bin/uglifyjs for Windows

Moved test input files to test/input.

Added test for #1236

Fixed sourceMapIncludeSources and inSourceMap = string combination of the UglifyJS.minify function.

Fix unneeded parens around unary args in new expression.

Test reparsing test/compress/*.js output

Legacy octal integer strict mode fixes

Build with AppVeyor on windows

Fix the document of keep_fnames option

Source map URL override from programmatic API

lib/sourcemap.js: Copy sourceContent from old souce-map to the new source-map. Should fix #882

Keep const in own scope while compressing

- Fixes #1205
- Fix provided by @kzc

Fix mangle with option keep_fnames=true for Safari.

Fixes: #1202

Fix error style for regex errors

Backport mocha with test from harmony

v2.7.0

Don't convert all strings to directives from moz-ast

Introduce a test that tests the --self build

Document that the smallest sequences optimization length is 2

and a sequences value of 1 is considered to be `true` - which
will be set to the default value of 200.

Change the default sequences limit to 200 to speed up compress.

Has little or no impact on minification size in the majority of
cases but can speed up rollup builds significantly.

This sequences change also has the beneficial side effect of avoiding
"stack size exceeded" errors on very large input files.

The user is free to alter the sequences limit if they are so inclined.
The previous sequences limit was 2000. 20 is often sufficient.