Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lombok - IllegalArgumentException in NotJavadoc starting in 2.21.0 #4034

Closed
magicprinc opened this issue Aug 1, 2023 · 5 comments
Closed
Labels

Comments

@magicprinc
Copy link

magicprinc commented Aug 1, 2023

Gradle 7.6.2
Build time: 2023-06-30 15:42:51 UTC
Revision: dab132169006b16e7ada4ab2456e0c9d6415b52a
Kotlin: 1.7.10
Groovy: 3.0.13
Ant: Apache Ant(TM) version 1.10.11 compiled on July 10 2021
JVM: 17.0.8 (BellSoft 17.0.8+7-LTS)
OS: Windows 10 10.0 amd64

lombok.config
lombok.addLombokGeneratedAnnotation = true

options.errorprone {
enabled = true
disableWarningsInGeneratedCode = true
excludedPaths = "./(generated|test)./."
disable("ParameterName")
disable("UnusedVariable")
disable("MissingSummary")
errorproneArgs = ["-XepExcludedPaths:.
/test/.*"]
}

net\sf\apr\auth\AccessDeniedException.java:1: error: An unhandled exception was thrown by the Error Prone static analysis plugin.
package net.sf.apr.auth;
^
Please report this at https://github.com/google/error-prone/issues/new and include the following:

 error-prone version: 2.21.0
 BugPattern: NotJavadoc
 Stack Trace:
 java.lang.IllegalArgumentException: Overlapping ranges not permitted but found [541..559] overlapping [541..559]
at com.google.common.base.Preconditions.checkArgument(Preconditions.java:445)
at com.google.common.collect.ImmutableRangeSet$Builder.build(ImmutableRangeSet.java:807)
at com.google.errorprone.bugpatterns.BugChecker.suppressedRegions(BugChecker.java:321)
at com.google.errorprone.bugpatterns.javadoc.NotJavadoc.matchCompilationUnit(NotJavadoc.java:62)
at com.google.errorprone.scanner.ErrorProneScanner.processMatchers(ErrorProneScanner.java:449)
at com.google.errorprone.scanner.ErrorProneScanner.visitCompilationUnit(ErrorProneScanner.java:555)
at com.google.errorprone.scanner.ErrorProneScanner.visitCompilationUnit(ErrorProneScanner.java:150)
at jdk.compiler/com.sun.tools.javac.tree.JCTree$JCCompilationUnit.accept(JCTree.java:614)
at jdk.compiler/com.sun.source.util.TreePathScanner.scan(TreePathScanner.java:60)
at com.google.errorprone.scanner.Scanner.scan(Scanner.java:58)
at com.google.errorprone.scanner.ErrorProneScannerTransformer.apply(ErrorProneScannerTransformer.java:43)
at com.google.errorprone.ErrorProneAnalyzer.finished(ErrorProneAnalyzer.java:156)
at jdk.compiler/com.sun.tools.javac.api.MultiTaskListener.finished(MultiTaskListener.java:132)
at jdk.compiler/com.sun.tools.javac.main.JavaCompiler.flow(JavaCompiler.java:1394)
at jdk.compiler/com.sun.tools.javac.main.JavaCompiler.flow(JavaCompiler.java:1341)
at jdk.compiler/com.sun.tools.javac.main.JavaCompiler.compile(JavaCompiler.java:933)
at jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.lambda$doCall$0(JavacTaskImpl.java:104)
at jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.invocationHelper(JavacTaskImpl.java:152)
at jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.doCall(JavacTaskImpl.java:100)
at jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.call(JavacTaskImpl.java:94)
at org.gradle.internal.compiler.java.IncrementalCompileTask.call(IncrementalCompileTask.java:92)
at org.gradle.api.internal.tasks.compile.AnnotationProcessingCompileTask.call(AnnotationProcessingCompileTask.java:94)
at org.gradle.api.internal.tasks.compile.ResourceCleaningCompilationTask.call(ResourceCleaningCompilationTask.java:57)
at org.gradle.api.internal.tasks.compile.JdkJavaCompiler.execute(JdkJavaCompiler.java:55)
at org.gradle.api.internal.tasks.compile.JdkJavaCompiler.execute(JdkJavaCompiler.java:39)
at org.gradle.api.internal.tasks.compile.daemon.AbstractDaemonCompiler$CompilerWorkAction.execute(AbstractDaemonCompiler.java:135)
at org.gradle.workers.internal.DefaultWorkerServer.execute(DefaultWorkerServer.java:63)
at org.gradle.workers.internal.AbstractClassLoaderWorker$1.create(AbstractClassLoaderWorker.java:49)
at org.gradle.workers.internal.AbstractClassLoaderWorker$1.create(AbstractClassLoaderWorker.java:43)
at org.gradle.internal.classloader.ClassLoaderUtils.executeInClassloader(ClassLoaderUtils.java:100)
at org.gradle.workers.internal.AbstractClassLoaderWorker.executeInClassLoader(AbstractClassLoaderWorker.java:43)
at org.gradle.workers.internal.FlatClassLoaderWorker.run(FlatClassLoaderWorker.java:32)
at org.gradle.workers.internal.FlatClassLoaderWorker.run(FlatClassLoaderWorker.java:22)
at org.gradle.workers.internal.WorkerDaemonServer.run(WorkerDaemonServer.java:87)
at org.gradle.workers.internal.WorkerDaemonServer.run(WorkerDaemonServer.java:56)
at org.gradle.process.internal.worker.request.WorkerAction$1.call(WorkerAction.java:138)
at org.gradle.process.internal.worker.child.WorkerLogEventListener.withWorkerLoggingProtocol(WorkerLogEventListener.java:41)
at org.gradle.process.internal.worker.request.WorkerAction.run(WorkerAction.java:135)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:36)
at org.gradle.internal.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:182)
at org.gradle.internal.remote.internal.hub.MessageHubBackedObjectConnection$DispatchWrapper.dispatch(MessageHubBackedObjectConnection.java:164)
at org.gradle.internal.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:414)
at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:49)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
@magicprinc
Copy link
Author

Same problem in other classes. Suddenly, completely unexpected (didn't change anything)

It seems that

  • disable("NotJavadoc")
    helps

@cushon
Copy link
Collaborator

cushon commented Aug 1, 2023

I suspect this is related to Lombok, perhaps that's why other checks including UnusedVariable were already disabled in that build config

Can you share a self-contained example that reproduces the crash?

@cushon cushon added the lombok label Aug 2, 2023
@magicprinc
Copy link
Author

@cushon Yes, definitely, I use Lombok

lombok.config file has setting:
lombok.addLombokGeneratedAnnotation = true

build.gradle
id("io.freefair.lombok") version "6.+" // -or- id 'io.franzbecker.gradle-lombok' version 'latest.release'

lombok {
version = 'latest.release'
}

I will try to reproduce this in small example, but it is hard, you know :-(

@magicprinc
Copy link
Author

The only thing that has changed, I can think of, is java version 17.0.7 → 17.0.8

@cushon cushon changed the title Error-Prone breaks build Lombok - IllegalArgumentException in NotJavadoc starting in 2.21.0 Aug 2, 2023
DenuxPlays added a commit to jasonlessenich/DIH4JDA that referenced this issue Aug 3, 2023
@cushon
Copy link
Collaborator

cushon commented Aug 4, 2023

Fixed by 2533218

@cushon cushon closed this as completed Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants