Skip to content

Commit 7e96318

Browse files
author
Vicente Romero
committedJun 23, 2021
8254571: Erroneous generic type inference in a lambda expression with a checked exception
Reviewed-by: jlahoda, mcimadamore
1 parent 1b2147a commit 7e96318

File tree

2 files changed

+52
-0
lines changed

2 files changed

+52
-0
lines changed
 

‎src/jdk.compiler/share/classes/com/sun/tools/javac/comp/DeferredAttr.java

+2
Original file line numberDiff line numberDiff line change
@@ -1217,6 +1217,7 @@ public void visitLambda(JCLambda tree) {
12171217
freeArgVars.nonEmpty()) {
12181218
stuckVars.addAll(freeArgVars);
12191219
depVars.addAll(inferenceContext.freeVarsIn(descType.getReturnType()));
1220+
depVars.addAll(inferenceContext.freeVarsIn(descType.getThrownTypes()));
12201221
}
12211222
scanLambdaBody(tree, descType.getReturnType());
12221223
}
@@ -1238,6 +1239,7 @@ public void visitReference(JCMemberReference tree) {
12381239
tree.getOverloadKind() != JCMemberReference.OverloadKind.UNOVERLOADED) {
12391240
stuckVars.addAll(freeArgVars);
12401241
depVars.addAll(inferenceContext.freeVarsIn(descType.getReturnType()));
1242+
depVars.addAll(inferenceContext.freeVarsIn(descType.getThrownTypes()));
12411243
}
12421244
}
12431245

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
/*
2+
* Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
3+
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4+
*
5+
* This code is free software; you can redistribute it and/or modify it
6+
* under the terms of the GNU General Public License version 2 only, as
7+
* published by the Free Software Foundation.
8+
*
9+
* This code is distributed in the hope that it will be useful, but WITHOUT
10+
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
11+
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
12+
* version 2 for more details (a copy is included in the LICENSE file that
13+
* accompanied this code).
14+
*
15+
* You should have received a copy of the GNU General Public License version
16+
* 2 along with this work; if not, write to the Free Software Foundation,
17+
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
18+
*
19+
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
20+
* or visit www.oracle.com if you need additional information or have any
21+
* questions.
22+
*/
23+
24+
/*
25+
* @test
26+
* @bug 8254571
27+
* @summary Erroneous generic type inference in a lambda expression with a checked exception
28+
* @compile ConsiderExceptionTVarsInStuckExprs.java
29+
*/
30+
31+
class ConsiderExceptionTVarsInStuckExprs {
32+
33+
public static void test() {
34+
outer(nested(x -> mightThrow()));
35+
outer(nested(ConsiderExceptionTVarsInStuckExprs::mightThrow2));
36+
}
37+
38+
static <A> void outer(Object o) {}
39+
40+
static <B, C, E extends Throwable> B nested(Fun<C,E> fun) {
41+
return null;
42+
}
43+
44+
interface Fun<X, Y extends Throwable> {
45+
void m(X t) throws Y;
46+
}
47+
48+
static void mightThrow() throws Exception {}
49+
static <C> void mightThrow2(C c) throws Exception {}
50+
}

0 commit comments

Comments
 (0)
Please sign in to comment.