StringUtil.stripQuotesAroundValue deprecated; StringUtil.unquoteString implementation...
authorSergey Simonchik <sergey.simonchik@jetbrains.com>
Sun, 11 Jan 2015 11:44:39 +0000 (14:44 +0300)
committerSergey Simonchik <sergey.simonchik@jetbrains.com>
Sun, 11 Jan 2015 11:44:39 +0000 (14:44 +0300)
platform/util/src/com/intellij/openapi/util/text/StringUtil.java

index f95039d7b40aa24fb070c89ab2bb1a6f7b0a594c..efb174a77b73b12ade7e0f36a37810570e189335 100644 (file)
@@ -613,24 +613,32 @@ public class StringUtil extends StringUtilRt {
     return buffer.toString();
   }
 
     return buffer.toString();
   }
 
+  private static boolean isQuoteAt(@NotNull String s, int ind) {
+    char ch = s.charAt(ind);
+    return ch == '\'' || ch == '\"';
+  }
+
+  @Contract(pure = true)
+  public static boolean isQuotedString(@NotNull String s) {
+    return s.length() > 1 && isQuoteAt(s, 0) && s.charAt(0) == s.charAt(s.length() - 1);
+  }
+
   @NotNull
   @Contract(pure = true)
   public static String unquoteString(@NotNull String s) {
   @NotNull
   @Contract(pure = true)
   public static String unquoteString(@NotNull String s) {
-    char c;
-    if (s.length() <= 1 || (c = s.charAt(0)) != '"' && c != '\'' || s.charAt(s.length() - 1) != c) {
-      return s;
+    if (isQuotedString(s)) {
+      return s.substring(1, s.length() - 1);
     }
     }
-    return s.substring(1, s.length() - 1);
+    return s;
   }
 
   @NotNull
   @Contract(pure = true)
   public static String unquoteString(@NotNull String s, char quotationChar) {
   }
 
   @NotNull
   @Contract(pure = true)
   public static String unquoteString(@NotNull String s, char quotationChar) {
-    char c;
-    if (s.length() <= 1 || (c = s.charAt(0)) != quotationChar || s.charAt(s.length() - 1) != c) {
-      return s;
+    if (s.length() > 1 && quotationChar == s.charAt(0) && quotationChar == s.charAt(s.length() - 1)) {
+      return s.substring(1, s.length() - 1);
     }
     }
-    return s.substring(1, s.length() - 1);
+    return s;
   }
 
   /**
   }
 
   /**
@@ -1487,8 +1495,15 @@ public class StringUtil extends StringUtilRt {
     return builder.toString();
   }
 
     return builder.toString();
   }
 
+  /**
+   * Strips quotes around the value.
+   * Quotes are removed even if leading and trailing quotes are different or if there is only one quote (leading or trailing).
+   * @deprecated use {@link com.intellij.openapi.util.text.StringUtil#unquoteString(String)} instead
+   * To be removed in IDEA 17
+   */
   @NotNull
   @Contract(pure = true)
   @NotNull
   @Contract(pure = true)
+  @Deprecated
   public static String stripQuotesAroundValue(@NotNull String text) {
     final int len = text.length();
     if (len > 0) {
   public static String stripQuotesAroundValue(@NotNull String text) {
     final int len = text.length();
     if (len > 0) {
@@ -1501,18 +1516,6 @@ public class StringUtil extends StringUtilRt {
     return text;
   }
 
     return text;
   }
 
-  private static boolean isQuoteAt(@NotNull String text, int ind) {
-    char ch = text.charAt(ind);
-    return ch == '\'' || ch == '\"';
-  }
-
-  @Contract(pure = true)
-  public static boolean isQuotedString(@NotNull String text) {
-    if (text.length() < 2) return false;
-    return startsWithChar(text, '\"') && endsWithChar(text, '\"')
-           || startsWithChar(text, '\'') && endsWithChar(text, '\'');
-  }
-
   /**
    * Formats the specified file size as a string.
    *
   /**
    * Formats the specified file size as a string.
    *