8305672: Surprising definite assignment error after JDK-8043179

Reviewed-by: kcr, vromero
This commit is contained in:
Archie Cobbs 2023-05-01 13:23:37 +00:00 committed by Vicente Romero
parent b39a9bffda
commit d437c61f5b
2 changed files with 49 additions and 0 deletions

View File

@ -2859,6 +2859,7 @@ public class Flow {
@Override
public void visitLambda(JCLambda tree) {
final Bits prevUninits = new Bits(uninits);
final Bits prevUninitsTry = new Bits(uninitsTry);
final Bits prevInits = new Bits(inits);
int returnadrPrev = returnadr;
int nextadrPrev = nextadr;
@ -2886,6 +2887,7 @@ public class Flow {
finally {
returnadr = returnadrPrev;
uninits.assign(prevUninits);
uninitsTry.assign(prevUninitsTry);
inits.assign(prevInits);
pendingExits = prevPending;
nextadr = nextadrPrev;

View File

@ -0,0 +1,47 @@
/*
* Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/
/*
* @test
* @bug 8305672
* @summary DA/DU tracking gets confused by lambda/try combination
* @compile TryWithLambdaFinal.java
*/
class TryWithLambdaFinal {
private final int x;
public TryWithLambdaFinal() {
try {
Runnable r = () -> {
try {
return;
} catch (Exception e) {
throw new AssertionError(e);
}
};
} catch (Exception e) {
}
x = 42; // this should be allowed
}
}