From 3adba2fa5d547d2e8616b252719ff28ae1977f11 Mon Sep 17 00:00:00 2001 From: Maxim Medvedev Date: Wed, 25 May 2011 15:06:25 +0400 Subject: [PATCH] fix test --- .../jetbrains/plugins/groovy/lang/psi/util/PsiUtil.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/PsiUtil.java b/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/PsiUtil.java index 3d9a8e109485..cebe522ecd3f 100644 --- a/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/PsiUtil.java +++ b/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/PsiUtil.java @@ -57,6 +57,7 @@ import org.jetbrains.plugins.groovy.lang.psi.api.statements.arguments.GrArgument import org.jetbrains.plugins.groovy.lang.psi.api.statements.arguments.GrArgumentList; import org.jetbrains.plugins.groovy.lang.psi.api.statements.arguments.GrNamedArgument; import org.jetbrains.plugins.groovy.lang.psi.api.statements.blocks.GrClosableBlock; +import org.jetbrains.plugins.groovy.lang.psi.api.statements.blocks.GrOpenBlock; import org.jetbrains.plugins.groovy.lang.psi.api.statements.branch.GrAssertStatement; import org.jetbrains.plugins.groovy.lang.psi.api.statements.branch.GrReturnStatement; import org.jetbrains.plugins.groovy.lang.psi.api.statements.branch.GrThrowStatement; @@ -1240,6 +1241,12 @@ public class PsiUtil { parent instanceof GrVariable) { return true; } - return ControlFlowUtils.collectReturns(ControlFlowUtils.findControlFlowOwner(expr), true).contains(expr); + final GrControlFlowOwner controlFlowOwner = ControlFlowUtils.findControlFlowOwner(expr); + if (controlFlowOwner instanceof GrOpenBlock && + controlFlowOwner.getParent() instanceof PsiMethod && + ((PsiMethod)controlFlowOwner.getParent()).getReturnType() == PsiType.VOID) { + return false; + } + return ControlFlowUtils.collectReturns(controlFlowOwner, true).contains(expr); } }