Optimize seq,void 0.  Close #377.

    (x, void 0)    => void x
    (x, undefined) => void x

Better fix for comments in AST_Exit

Close #374

Take out all comments from an AST_Exit's value

Fix #372

v2.4.8

Add option to adjust the src/target line in the source map

minor

Merge pull request #371 from colorhook/master

bugfix #242

bugfix #242

semicolons

Merge pull request #245 from ForbesLindesay/patch-1

Make `DefaultsError` a real `Error` object

Add `drop_console` option to the compressor

minor

v2.4.7

AST_Catch shouldn't really inherit from AST_Scope.  Fix #363

I hereby acknowledge that figure_out_scope has become a mess.

Properly scope `catch` identifier when --screw-ie8

Fix #344

v2.4.6, because npm is foobar

v2.4.5

Add --noerr to turn off argument name checking

for now only used for keys passed to `-c` or `-b`.

v2.4.4

Fix faulty compression

`String(x + 5)` is not always the same as `x + "5"`.  Overlooked that. :-(

Close #350

Only descend twice after drop_unused if it's the same node type.

Fix #345

Better fix for #343

We can in fact lift sequences, but only if the operation is assignment and
the left-hand side has no side effects nor property access -- that should
guarantee that whatever we place before it cannot affect the sense of the
assignment.

Dropped contrived test case (too hard to support it now), added a more
meaningful one.

v2.4.3

Do not lift sequence from right-hand side of binary operation.  Fix #343

v2.4.2

[README] Fix #278

Workaround for Safari bug

Close #313

Fix parsing setters/getters (allow keywords for name).

The "key" property was always "set" or "get", which didn't make much sense.
Now it'll be the actual name of the setter/getter (AST_Node), and the
AST_Accessor object itself, which represents the function, won't store any
name.

Close #319

Fix regression after e4c530240650535d1cb46569dfb013193471af05

`x * (y * z)` ==> `x * y * z` -- the better place to do this is in the
compressor rather than codegen.

indentation

Fix output for `x = 2 * (a % b / b * c)`

(issue #337)

minor

Disallow reversal where lhs has higher or equal precedence

Fixes #267

Fix typo

Fix reading arguments

i.e. read `-c unsafe,unsafe-comps` as `-c unsafe=true,unsafe_comps=true`

Avoid shadowing name of function expression with function argument

Close #179, #326, #327

Add "preamble" output option

Close #335

Fix codegen for when comments_before is undefined.

Fix #333

Merge branch 'master' of github.com:mishoo/UglifyJS2

Display number of failed tests and corresponding files

Merge pull request #330 from markjaquith/master

Unit test to detect issue in 8d14efe for #126 that causes aggressive parenthesis removal, functional differences

Merge pull request #331 from rvanvelzen/rhs-strings-fix

Fix RHS concat (raised in #330)

Fix RHS concat (raised in #330)

When attempting to concat the left-side of the rhs, make sure the rhs is
a string.

Add a unit test for issue-126

Add a unit test to test to test for aggressive parenthesis removal that causes functional changes.

Merge pull request #325 from rvanvelzen/fix-269

Fix #269

Fix #269

Shorten most primitives where possible. Also optimize some edge cases.

Merge pull request #323 from rvanvelzen/undefined-drop-vars-fix

Fix #280

Fix #280

The `init` of the `ForStatement` is not a `BlockStatement` before it was
descended. The descend has to happen first, and *then* the actual
checks.

Merge pull request #322 from rvanvelzen/test-exit-code-1

Add an exit code to the test suite

Add an exit code to the test suite

By adding the exit code 1 (or any other non-zero exit code) `npm test`
will know the tests didn't perform correctly. This way it's easier to
know if pull requests are good or bad.

Revert #3a81f60 for now

(with it some tests break and it can generate invalid output, see issue #44)

v2.4.1

`arguments` outside of a function is an ordinary variable.

Fix #501

wrap up

Don't drop_unused before compression.

Fix #280, #282

Rename clean_getters to pure_getters; add pure_funcs.

Merge pull request #308 from meteor/fix-unicode-keys

Only allow identifier start characters at the beginning of identifiers.

Only allow identifier start characters at the beginning of identifiers.

Without this fix, the following source:

   x = {"\u200c": 42};

would incorrectly be converted into a quoteless key. But while \u200c is allowed
to be in identifiers, it cannot be at the beginning, as per ES5.

(For example, the SockJS client library doesn't work under uglify starting with
d9ad3c7c.)

add `clean_getters` compressor option (default `false`)

allows one to specify if `foo.bar` is considered to have side effects.

minor optimization

unlikely to help in hand-written code:

    (something() ? foo : bar) == foo  ==>  something()

Fix error in the output minifying `Function("return this")()`

More attempts to determine when addition is associative

Somebody hit me with bug reports on this. :)

Refs #300

Actually let's move away those monsters from the evaluate function

ev() should do a single thing — evaluate constant expressions.  if that's
not possible, just return the original node.  it's not the best place for
partial evaluation there, instead doing it in the compress functions.

More dirty handling of [ ... ].join() in unsafe mode

Close #300

Evaluate [...].join() if possible: minor bugfix

Follow-up to 78e98d2.

When `unsafe` is set, evaluate [...].join() if possible

Close #298

Concatenate strings also on the right-hand side of an expression that cannot be evaluated. Fix #126

E.g. converts:
  a+'Hello'+'World'
to
  a+'HelloWorld'

Avoid printing <!-- in the output (HTML5 comment)

Support HTML5 comment syntax (enabled by default!)

See http://javascript.spec.whatwg.org/#comment-syntax
    https://github.com/mishoo/UglifyJS/issues/503
    https://github.com/marijnh/acorn/issues/62

minor

Fix names.

Disallow `continue` referring to a non-IterationStatement.  Fix #287

Simplifies handling of labels (their definition/references can be easily
figured out at parse time, no need to do it in `figure_out_scope`).

Better fix for #286

Fix parsing `a.case /= 1`

Close #286

Fix parsing regexp after unary-prefix operator

++/x/.y

Fix #284

Better reporting of parse errors

v2.4.0

Remove --ie-proof from the readme.

Fix #276

Move support for `negate_iife` in the compressor, rather than code generator

(the code generator doesn't maintain enough context to know whether
the return value is important or discarded)

Fixes #272

Merge pull request #270 from michaelficarra/GH-259

fixes #259: don't unnecessarily quote object properties when --screw-ie8

fixes #259: don't unnecessarily quote object properties when --screw-ie8

fix usage

Don't drop unused setter argument.

Fix #257

Support `-p relative`.  Fix #256

Add support for browserify

Don't pretend to evaluate lambdas

Fix #255

Fix #251

Don't require arguments to --enclose

Reverting "added option for dropping unused params"
Revert "added option for dropping unused params"

(turns out we already had the `unused` option for this.)

This reverts commit e54df2226f7f3887d2f850cea8caf5c0353dce00.

added option for dropping unused params

Make `DefaultsError` a real `Error` object

Fix output for arrays whose last element is a hole: [1,,]

1529ab96 started to do this (by considering holes to be separate from
"undefined") but it still converted
   [1,,]    (length 2, last element hole, trailing comma)
to
   [1,]     (length 1, trailing comma)

Unfortunately the test suite doesn't really make this clear: the new test here
passes with or without this patch because run-tests.js beautifys the expected
output (in "make_code"), which does the incorrect transformation! If you make
some manual change to arrays.js to make the test fail and see the INPUT and
OUTPUT, then you can see that without this fix, [1,,] -> [1,], and with this fix
it stays [1,,].

Revert previous patch, it was no good.

Add "position" option to parser, to specify initial pos/line/col

(for parsing embedded scripts)

Apply transformer to AST_VarDef's name

Fix #237

Fix typo.

Close #239

Reset the base54 counters every time minify is called.

Close #229

Don't swap binary ops when "use asm" is in effect.

Refs #167

Merge pull request #220 from lautis/escape-null

Escape null characters as \x00

Escape null characters as \x00

Since \0 might be mistakenly interpreted as octal if followed by a
number and using literal null is in some cases interpreted as end of
string, escape null as \x00.