[performance] less calls to isInModuleDeclaration() #1747

Reorder costly conditions #1747
eclipse-jdt · Jan 10, 2024 · 6170f1e · 6170f1e
1 parent 0a573e1
commit 6170f1e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/...eclipse.jdt.core.compiler.batch/src/org/eclipse/jdt/internal/compiler/parser/Scanner.java b/...eclipse.jdt.core.compiler.batch/src/org/eclipse/jdt/internal/compiler/parser/Scanner.java
@@ -4902,8 +4902,9 @@ private VanguardScanner getNewVanguardScanner() {
 	return vs;
 }
 protected final boolean mayBeAtCasePattern(int token) {
-	return (!isInModuleDeclaration() && JavaFeature.PATTERN_MATCHING_IN_SWITCH.isSupported(this.complianceLevel, this.previewEnabled))
-			&& (token == TokenNamecase || this.multiCaseLabelComma);
+	return ((token == TokenNamecase || this.multiCaseLabelComma)
+			&& JavaFeature.PATTERN_MATCHING_IN_SWITCH.isSupported(this.complianceLevel, this.previewEnabled))
+			&& !isInModuleDeclaration();
 }
 protected final boolean maybeAtLambdaOrCast() { // Could the '(' we saw just now herald a lambda parameter list or a cast expression ? (the possible locations for both are identical.)