Issue checkstyle#10924: Refactor NoWhitespaceBeforeCheck

MUzairS15 · Mar 16, 2022 · 2aa0429 · 2aa0429
1 parent 7697df3
commit 2aa0429
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/NoWhitespaceBeforeCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/NoWhitespaceBeforeCheck.java
@@ -23,6 +23,7 @@
 import com.puppycrawl.tools.checkstyle.api.AbstractCheck;
 import com.puppycrawl.tools.checkstyle.api.DetailAST;
 import com.puppycrawl.tools.checkstyle.api.TokenTypes;
+import com.puppycrawl.tools.checkstyle.utils.CodePointUtil;
 import com.puppycrawl.tools.checkstyle.utils.CommonUtil;
 
 /**
@@ -209,19 +210,15 @@ public int[] getRequiredTokens() {
 
     @Override
     public void visitToken(DetailAST ast) {
-        final String line = getLine(ast.getLineNo() - 1);
+        final int[] line = getLineCodePoints(ast.getLineNo() - 1);
         final int before = ast.getColumnNo() - 1;
-        final int[] codePoints = line.codePoints().toArray();
 
-        if ((before == -1 || CommonUtil.isCodePointWhitespace(codePoints, before))
+        if ((before == -1 || CommonUtil.isCodePointWhitespace(line, before))
                 && !isInEmptyForInitializerOrCondition(ast)) {
             boolean flag = !allowLineBreaks;
             // verify all characters before '.' are whitespace
-            for (int i = 0; i <= before - 1; i++) {
-                if (!CommonUtil.isCodePointWhitespace(codePoints, i)) {
-                    flag = true;
-                    break;
-                }
+            if (before > -1 && !CodePointUtil.hasWhitespaceBefore(before, line)) {
+                flag = true;
             }
             if (flag) {
                 log(ast, MSG_KEY, ast.getText());