minor improvement to string optimisation (#1514)

- "" + "a"     => "a"
- "" + a + "b" => a + "b"
- "a" + ""     => "a" (improving on #45)

v2.8.4

fix corner cases in `reduce_vars` (#1524)

Avoid variable substitution in the following cases:
- use of variable before declaration
- declaration within conditional code blocks
- declaration within loop body

fixes #1518
fixes #1525

v2.8.3

fix crash on missing `props` to `string_template()` (#1523)

Patched up `make_node()` without `orig`.

There may be other cases where `start` could be missing, so make it print "undefined" instead of crashing.

fixes #1518

v2.8.2

invert `reduce_vars` tracking flag (#1519)

Modules like webpack and grunt-contrib-uglify still uses `ast.transform(compressor)` before `Compressor.compress(ast)` was introduced.

Workaround this compatibility issue by deactivating `reduce_vars` in such case.

Also fix use case with omitted `options` when calling `Compressor()`.

fixes #1516

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