Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build (#393) 

* LUCENE-10185: pass --release 11 to ECJ linter, fix JDK 17 build

Otherwise, new java releases such as JDK 18, JDK 19, ... may have even
more new deprecations, the build shouldn't fail in such cases.

Remove -source/-target now that we pass --release

Fix casting so ECJ understands it and creates correct call signature (UweSays: "It's ok. I know why it happens, but it's a bug in ECJ. The type safety is checked by the invokeexact")

Co-authored-by: Uwe Schindler <uschindler@apache.org>
This commit is contained in:
Robert Muir 2021-10-18 16:43:53 -04:00 committed by GitHub
parent c4c3c3270e
commit f8d431ae44
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
gradle/validation/ecj-lint.gradle
View File

@ -63,8 +63,7 @@ allprojects {
args += [ "-d", "none" ]
// Compilation environment.
args += [ "-source", project.java.sourceCompatibility ]
args += [ "-target", project.java.targetCompatibility ]
args += [ "--release", project.java.targetCompatibility ]
args += [ "-encoding", "UTF-8"]
args += [ "-proc:none" ]
args += [ "-nowarn" ]

6
lucene/core/src/java/org/apache/lucene/util/AttributeFactory.java
View File

@ -149,9 +149,13 @@ public abstract class AttributeFactory {
final MethodHandle constr = findAttributeImplCtor(clazz);
return new StaticImplementationAttributeFactory<A>(delegate, clazz) {
@Override
@SuppressWarnings("unchecked")
protected A createInstance() {
try {
return (A) constr.invokeExact();
// be explicit with casting, so javac compiles correct call to polymorphic signature:
final AttributeImpl impl = (AttributeImpl) constr.invokeExact();
// now cast to generic type:
return (A) impl;
} catch (Error | RuntimeException e) {
throw e;
} catch (Throwable e) {