Wow - nice. Far simpler implementation than the dump AST patch.

Wow - nice. Far simpler implementation than the dump AST patch.

I cheat by utilising the recursive nature of JSON.stringify()'s replacer 👻

This will make debugging infinitely easier.

@kzc reading your patch at #769 (comment) again, I noticed I've missed reordering of AST_Binary's properties.

I noticed I've missed reordering of AST_Binary's properties.

Thanks. Makes it much easier to read the AST output.

Something is not quite right with the latest version of this PR:

$ bin/uglifyjs --self -mc passes=3 -o ast
bin/uglifyjs:380
    return skip_keys.indexOf(key) >= 0;
                    ^

TypeError: Cannot read property 'indexOf' of undefined

Would you mind adding at least one AST dump mocha test?

This is a workaround for the bug above:

bin/uglifyjs --self -mc passes=3 | bin/uglifyjs -o ast

That's one strange bug indeed - let me investigate, then put in a quick mocha test.

Edit: yeah, I'm being an idiot...

Something is not quite right with the latest version of this PR:

Should be fixed by 1040110, alongside a mocha test.

Odd - why would moving the skip_keys array to the top fix the problem for some cases?

Reading from process.stdin is asynchronous, i.e. run() gets called within the event loop, which means the rest of bin/uglifyjs gets to be evaluated before it.

The previous version of this PR also works because it does not contain var skip_keys, so there wasn't any issue with execution order.

Reading from process.stdin is asynchronous

Thanks. It's obvious in hindsight.

This feature works great - best thing since sliced bread.