RFR: 8282574: Cleanup unnecessary calls to Throwable.initCause() in jdk.compiler
Joe Darcy
darcy at openjdk.java.net
Fri Mar 4 06:51:03 UTC 2022
On Wed, 2 Mar 2022 18:43:35 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:
> Pass cause exception as constructor parameter is shorter and easier to read.
Marked as reviewed by darcy (Reviewer).
src/jdk.compiler/share/classes/com/sun/tools/javac/processing/JavacProcessingEnvironment.java line 303:
> 301: processorIterator = List.of(new PrintingProcessor()).iterator();
> 302: } catch (Throwable t) {
> 303: throw new AssertionError("Problem instantiating PrintingProcessor.", t);
When this annotation processing code was written in JDK 6, I wondered why we didn't use the other constructor at the time. Turns out, that constructor of AssertionError was only added in JDK 7 under [JDK-6935997](https://bugs.openjdk.java.net/browse/JDK-6935997) :-)
-------------
PR: https://git.openjdk.java.net/jdk/pull/7662
More information about the compiler-dev
mailing list