UsafeLocaleUsage: update the proposed fix to use replace(char, char)

The current `replace(CharSequence, CharSequence)` triggers a JavaOptionalSuggestions, so we might as well apply it directly.

PiperOrigin-RevId: 697528702

Author: java-team-github-bot

core/src/main/java/com/google/errorprone/bugpatterns/UnsafeLocaleUsage.java

@@ -84,10 +84,9 @@ public Description matchNewClass(NewClassTree tree, VisitorState state) {
         String replacementArg =
             arg instanceof JCLiteral
                 ? String.format(
-                    "\"%s\"", ASTHelpers.constValue(arg, String.class).replace("_", "-"))
+                    "\"%s\"", ASTHelpers.constValue(arg, String.class).replace('_', '-'))
                 : String.format(
-                    "%s.replace(\"_\", \"-\")",
-                    state.getSourceForNode(constructorArguments.get(0)));
+                    "%s.replace('_', '-')", state.getSourceForNode(constructorArguments.get(0)));
 
         descriptionBuilder.addFix(
             SuggestedFix.replace(tree, String.format("Locale.forLanguageTag(%s)", replacementArg)));

core/src/test/java/com/google/errorprone/bugpatterns/UnsafeLocaleUsageTest.java

@@ -60,7 +60,7 @@ class Test {
               static class Inner {
                 private Locale locale;
                 Inner(String a) {
-                   locale = Locale.forLanguageTag(a.replace("_", "-"));
+                   locale = Locale.forLanguageTag(a.replace('_', '-'));
                 }
               }