infinite deep clone in function inlining

[kzc]

Bug report or feature request?

bug

ES5 or ES6+ input?

both

Uglify version (uglifyjs -V)

3.1.7

JavaScript input

$ cat clone_bug.js 
!function() {
    function foo() {
        bar();
    }
    function bar() {
        foo();
    }
}();

$ bin/uglifyjs clone_bug.js -c
ERROR: Maximum call stack size exceeded

Any number of functions can be involved:

$ cat clone_bug2.js 
!function() {
    function foo() { bar(); }
    function bar() { qux(); }
    function qux() { foo(); }
}();

$ bin/uglifyjs clone_bug2.js -c
ERROR: Maximum call stack size exceeded

related issue: #2438

[kzc]

@alexlamsl If we mark AST_Lambda instances that have already been inlined that ought to take care of this.

[alexlamsl]

Thanks for isolating this – I now get to have a full head of hair for another decade.

Unfortunately I can only fix this later on this evening, so 3.1.8 will need to wait for (early) tomorrow.

[kzc]

Fixed in #2445
Fixed in #2446.