[BUG] java.lang.StackOverflowError when method has more then 500 lines of code

[djogss]

Describe the bug
Produce the java.lang.StackOverflowError if codebase contains the method longer than 500 lines

To Reproduce
Please see repository on github with instructions:
https://github.com/djogss/lombok-issue

Expected behavior
Code compiles without any issue

Version info (please complete the following information):

• Lombok version: 1.18.12
• Java version:

 openjdk version "15-ea" 2020-09-15
 OpenJDK Runtime Environment (build 15-ea+21-975)
 OpenJDK 64-Bit Server VM (build 15-ea+21-975, mixed mode, sharing)

The same happens on java 8 as well.

[Rawi01]

Stacktrace:

Caused by: java.lang.StackOverflowError
        at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:384)
        at lombok.javac.JavacAST.buildTree(JavacAST.java:196)
        at lombok.javac.JavacAST.buildTree(JavacAST.java:1)
        at lombok.core.AST.buildWithField0(AST.java:394)
        at lombok.core.AST.buildWithField(AST.java:292)
        at lombok.javac.JavacAST.drill(JavacAST.java:421)
        at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:387)
        at lombok.javac.JavacAST.buildTree(JavacAST.java:196)
        at lombok.javac.JavacAST.buildTree(JavacAST.java:1)
        at lombok.core.AST.buildWithField0(AST.java:394)
        at lombok.core.AST.buildWithField(AST.java:292)
        at lombok.javac.JavacAST.drill(JavacAST.java:421)
        at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:387)
        ...

The problem is the really long chained method call. As a workaround you can split it up into multiple shorter chains.

[randakar]

This is the example right? https://github.com/djogss/lombok-issue/blob/master/src/main/java/com/example/demo/springsecurity/WebSecurityConfig.java Damn. I can see how someone might arrive at something like that but .. really, there should be a way to just check a list of id's instead of doing it the painful way. Digression aside: I really wonder why lombok tries to parse that whole thing, the only annotation on there is `@Slf4j` ..

…

On Wed, May 6, 2020 at 11:43 PM Rawi01 ***@***.***> wrote: Stacktrace: Caused by: java.lang.StackOverflowError at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:384) at lombok.javac.JavacAST.buildTree(JavacAST.java:196) at lombok.javac.JavacAST.buildTree(JavacAST.java:1) at lombok.core.AST.buildWithField0(AST.java:394) at lombok.core.AST.buildWithField(AST.java:292) at lombok.javac.JavacAST.drill(JavacAST.java:421) at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:387) at lombok.javac.JavacAST.buildTree(JavacAST.java:196) at lombok.javac.JavacAST.buildTree(JavacAST.java:1) at lombok.core.AST.buildWithField0(AST.java:394) at lombok.core.AST.buildWithField(AST.java:292) at lombok.javac.JavacAST.drill(JavacAST.java:421) at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:387) ... The problem is the really long chained method call. As a workaround you can split it up into multiple shorter chains. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#2453 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABIERNL3ZE7MXGCIF46CQLRQHKZ5ANCNFSM4M2IHI6A> .

-- "Don't only practice your art, but force your way into it's secrets, for it and knowledge can raise men to the divine." -- Ludwig von Beethoven

[MichelMunoz]

Reproduced on linux, with eclipse, on edge and 1.18.12, when doing a gradle clean/build
And it get worse : this happens even if the code does not use lombok annotations at all ; the project is configured to use lombok, but the code - only one file, see below - doesn't use any annotation or lombok stuff ; on 1.18.12 it produces a:

Caused by: java.lang.StackOverflowError
	at lombok.core.AST.fieldsOf(AST.java:234)
	at lombok.javac.JavacAST.drill(JavacAST.java:421)
	at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:387)
	at lombok.javac.JavacAST.buildTree(JavacAST.java:196)
	at lombok.javac.JavacAST.buildTree(JavacAST.java:1)
	at lombok.core.AST.buildWithField0(AST.java:394)
	at lombok.core.AST.buildWithField(AST.java:292)
	at lombok.javac.JavacAST.drill(JavacAST.java:421)
	at lombok.javac.JavacAST.buildStatementOrExpression(JavacAST.java:387)
	at lombok.javac.JavacAST.buildTree(JavacAST.java:196)
	at lombok.javac.JavacAST.buildTree(JavacAST.java:1)
	at lombok.core.AST.buildWithField0(AST.java:394)
	at lombok.core.AST.buildWithField(AST.java:292)
	at lombok.javac.JavacAST.drill(JavacAST.java:421)
        ...

similar (but not identical) to the one listed by @Rawi01

FxPalettes.java.zip
The source just uses a builder-type construction of a color palette, but I imagine it can be generalized to any non-lombok using class with long chained calls of size at least 500.

[MichelMunoz]

I've made an even simpler, minimal, reproduction of the problem, can be used for unit test or something. Just some class that initializes a static field with a long chained call.
MinimalReproductionOfProblem.java.zip

[rspilker]

I'm closing all duplicates of #1118