Fixed flakyness of RegexInjectionRemediator

Author: andrecsilva

framework/codemodder-base/src/main/java/io/codemodder/remediation/DefaultFixCandidateSearcher.java

@@ -73,6 +73,9 @@ public FixCandidateSearchResults<T> search(
                                       n, issueStartLine, issueEndLine, column))
                           .orElse(nodePositionMatcher.match(n, issueStartLine, issueEndLine)))
               .toList();
+      System.out.println(issueStartLine);
+      System.out.println(maybeColumn);
+      System.out.println(nodesForIssue);
       if (nodesForIssue.isEmpty()) {
         continue;
       }

framework/codemodder-base/src/main/java/io/codemodder/remediation/regexinjection/RegexInjectionRemediator.java

@@ -63,12 +63,12 @@ public SuccessOrReason fix(final CompilationUnit cu, final Node node) {
     }
   }
 
-  /** Check if its a {@link Pattern#compile(String)} call. */
+  /** Check if it's a {@link Pattern#compile(String)} call. */
   private static boolean isCompileCall(final MethodCallExpr methodCallExpr) {
-    return methodCallExpr.getNameAsString().equals("compile")
-            && methodCallExpr.getArguments().size() == 1
-        || methodCallExpr.getArguments().size() == 2
-            && !methodCallExpr.getArguments().get(0).isStringLiteralExpr();
+    return "compile".equals(methodCallExpr.getNameAsString())
+        && (methodCallExpr.getArguments().size() == 1
+            || (methodCallExpr.getArguments().size() == 2
+                && !methodCallExpr.getArguments().get(0).isStringLiteralExpr()));
   }
 
   /** Check if its a {@link String#replaceFirst(String, String)} call. */