From 15dd2ecf6e8423dcc52210b08720fea0cd1e1561 Mon Sep 17 00:00:00 2001 From: Bas Leijdekkers Date: Thu, 27 Apr 2017 20:13:54 +0200 Subject: [PATCH] RegExp: replace switch statement with simple if --- .../EscapedMetaCharacterInspection.java | 24 ++++++------------- 1 file changed, 7 insertions(+), 17 deletions(-) diff --git a/RegExpSupport/src/org/intellij/lang/regexp/inspection/EscapedMetaCharacterInspection.java b/RegExpSupport/src/org/intellij/lang/regexp/inspection/EscapedMetaCharacterInspection.java index a8a9642e04f7..86a81cc65e1e 100644 --- a/RegExpSupport/src/org/intellij/lang/regexp/inspection/EscapedMetaCharacterInspection.java +++ b/RegExpSupport/src/org/intellij/lang/regexp/inspection/EscapedMetaCharacterInspection.java @@ -66,24 +66,14 @@ public class EscapedMetaCharacterInspection extends LocalInspectionTool { return; } final char c = text.charAt(1); - switch (c) { - case '{': - case '}': - case '(': - case ')': - case '.': - case '*': - case '+': - case '?': - case '|': - case '$': - final ASTNode node = ch.getNode().getFirstChildNode(); - if (node != null && node.getElementType() == RegExpTT.REDUNDANT_ESCAPE) { - return; - } - myHolder.registerProblem(ch, "Escaped meta character " + c + "", new EscapedMetaCharacterFix(c)); - break; + if ("{}().*+?|$".indexOf(c) < 0) { + return; } + final ASTNode node = ch.getNode().getFirstChildNode(); + if (node != null && node.getElementType() == RegExpTT.REDUNDANT_ESCAPE) { + return; + } + myHolder.registerProblem(ch, "Escaped meta character " + c + "", new EscapedMetaCharacterFix(c)); } }