vcs: fix i18n warnings - mark StringBuilders as Nls
authorAleksey Pivovarov <AMPivovarov@gmail.com>
Thu, 20 Aug 2020 14:13:14 +0000 (17:13 +0300)
committerintellij-monorepo-bot <intellij-monorepo-bot-no-reply@jetbrains.com>
Thu, 20 Aug 2020 14:14:01 +0000 (14:14 +0000)
GitOrigin-RevId: ec7c2eab996edaf815f7481ef7e69815cc4878fc

platform/dvcs-impl/src/com/intellij/dvcs/DvcsUtil.java
platform/dvcs-impl/src/com/intellij/dvcs/ui/CloneDvcsDialog.java
platform/vcs-impl/src/git4idea/annotate/AnnotationTooltipBuilder.java
plugins/git4idea/src/git4idea/GitVcs.java
plugins/hg4idea/src/org/zmlx/hg4idea/provider/annotate/HgAnnotation.java

index df57fcf517c5e09b96ecdd5a84542dd5e0b73421..bc314589de235d5f2b4c6a16cb22350ebb66d742 100644 (file)
@@ -449,7 +449,7 @@ public final class DvcsUtil {
     boolean isLimited = limit >= 2 && limit < size;
     int listCount = (isLimited ? limit : size) - 1;
 
-    StringBuilder sb = new StringBuilder();
+    @Nls StringBuilder sb = new StringBuilder();
     for (int i = 0; i < listCount; i++) {
       if (i != 0) sb.append(DvcsBundle.message("sequence.concatenation.separator"));
       sb.append(strings.get(i));
@@ -461,6 +461,6 @@ public final class DvcsUtil {
     else {
       sb.append(DvcsBundle.message("sequence.concatenation.tail", strings.get(size - 1)));
     }
-    return sb.toString(); //NON-NLS
+    return sb.toString();
   }
 }
index 02f9c148c4a830b7fb4b0407869de1da002334de..987d9bfe726fb05ad35be159889f493169468106 100644 (file)
@@ -284,11 +284,11 @@ public abstract class CloneDvcsDialog extends DialogWrapper {
         showRepositoryUrlAutoCompletionTooltip();
         if (!myErrors.isEmpty()) {
           for (RepositoryListLoadingException error : myErrors) {
-            StringBuilder errorMessageBuilder = new StringBuilder();
+            @Nls StringBuilder errorMessageBuilder = new StringBuilder();
             errorMessageBuilder.append(error.getMessage());
             Throwable cause = error.getCause();
             if (cause != null) errorMessageBuilder.append(": ").append(cause.getMessage());
-            @Nls String message = errorMessageBuilder.toString(); //NON-NLS
+            @Nls String message = errorMessageBuilder.toString();
             myRepositoryListLoadingErrors.add(new ValidationInfo(message).asWarning().withOKEnabled());
           }
           startTrackingValidation();
index bb8e95dff051e3ab9144d1959b861191c310784f..52874791c8ea6539edee641b286c0977104c4e18 100644 (file)
@@ -2,12 +2,14 @@
 package git4idea.annotate;
 
 import com.intellij.openapi.project.Project;
+import com.intellij.openapi.util.NlsSafe;
 import com.intellij.openapi.vcs.VcsBundle;
 import com.intellij.openapi.vcs.changes.issueLinks.IssueLinkHtmlRenderer;
 import com.intellij.openapi.vcs.history.VcsRevisionNumber;
 import com.intellij.util.containers.Convertor;
 import com.intellij.vcsUtil.VcsUtil;
 import com.intellij.xml.util.XmlStringUtil;
+import org.jetbrains.annotations.Nls;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
@@ -15,18 +17,18 @@ public class AnnotationTooltipBuilder {
   @NotNull private final Project myProject;
   private final boolean myAsHtml;
 
-  private final StringBuilder sb = new StringBuilder();
+  private final @Nls StringBuilder sb = new StringBuilder();
 
   public AnnotationTooltipBuilder(@NotNull Project project, boolean asHtml) {
     myProject = project;
     myAsHtml = asHtml;
   }
 
-  private void append(@NotNull String text) {
+  private void append(@NotNull @Nls String text) {
     sb.append(myAsHtml ? XmlStringUtil.escapeString(text) : text);
   }
 
-  private void appendRaw(@NotNull String text) {
+  private void appendRaw(@NotNull @Nls String text) {
     sb.append(text);
   }
 
@@ -47,17 +49,17 @@ public class AnnotationTooltipBuilder {
     appendRaw(VcsBundle.message("commit.description.tooltip.commit", revision));
   }
 
-  public void appendLine(@NotNull String content) {
+  public void appendLine(@NotNull @Nls String content) {
     appendNewline();
     append(content);
   }
 
-  public void appendCommitMessageBlock(@NotNull String message) {
+  public void appendCommitMessageBlock(@NotNull @Nls String message) {
     append("\n\n");
     appendTextWithLinks(message);
   }
 
-  public void appendTextWithLinks(@NotNull String message) {
+  public void appendTextWithLinks(@NotNull @Nls String message) {
     if (myAsHtml) {
       appendRaw(IssueLinkHtmlRenderer.formatTextWithLinks(myProject, message));
     }
@@ -70,6 +72,7 @@ public class AnnotationTooltipBuilder {
     if (sb.length() != 0) append("\n");
   }
 
+  @Nls
   @Override
   public String toString() {
     return sb.toString();
@@ -78,9 +81,9 @@ public class AnnotationTooltipBuilder {
   @NotNull
   public static String buildSimpleTooltip(@NotNull Project project,
                                           boolean asHtml,
-                                          @NotNull String prefix,
-                                          @NotNull String revision,
-                                          @Nullable String commitMessage) {
+                                          @NotNull @Nls String prefix,
+                                          @NotNull @NlsSafe String revision,
+                                          @Nullable @Nls String commitMessage) {
     AnnotationTooltipBuilder builder = new AnnotationTooltipBuilder(project, asHtml);
     builder.append(prefix);
     builder.append(" ");
index ea924590f4dbc72471b13c2d5029ad55c18377e5..ae0a79602f531a165ea0dab4dbfa2ab23e21eca0 100644 (file)
@@ -247,7 +247,7 @@ public final class GitVcs extends AbstractVcs {
    */
   public void showErrors(@NotNull List<? extends VcsException> list, @NotNull @Nls String action) {
     if (list.size() > 0) {
-      StringBuilder buffer = new StringBuilder();
+      @Nls StringBuilder buffer = new StringBuilder();
       buffer.append("\n");
       buffer.append(GitBundle.message("error.list.title", action));
       for (final VcsException exception : list) {
index a701098d79426c9018707dd8930cf139ee29240b..ce7b8951cf15e84495b9b05e23635777cf5ee6a9 100644 (file)
@@ -118,7 +118,7 @@ public class HgAnnotation extends FileAnnotation {
     atb.appendLine(HgBundle.message("hg4idea.annotation.date", revision.getRevisionDate()));
     String message = revision.getCommitMessage();
     if (message != null) atb.appendCommitMessageBlock(message);
-    return atb.toString(); //NON-NLS
+    return atb.toString();
   }
 
   @Override