Renamed SafeSubstring param and supporting logic

jerbri · jerbri · commit b34f6e41b132 · 2014-05-21T22:36:27.000-04:00
SafeSubstring was passing "length" as "startIndex" incorrectly.
diff --git a/src/ServiceStack.Text/StringExtensions.cs b/src/ServiceStack.Text/StringExtensions.cs
@@ -601,11 +601,9 @@ public static string ToLowercaseUnderscore(this string value)
             return sb.ToString();
         }
 
-        public static string SafeSubstring(this string value, int length)
+        public static string SafeSubstring(this string value, int startIndex)
         {
-            return String.IsNullOrEmpty(value)
-                ? String.Empty
-                : value.Substring(Math.Min(length, value.Length));
+            return SafeSubstring(value, startIndex, value.Length);
         }
 
         public static string SafeSubstring(this string value, int startIndex, int length)

Original file line number	Diff line number	Diff line change
`@@ -601,11 +601,9 @@ public static string ToLowercaseUnderscore(this string value)`
`601`	`601`	`return sb.ToString();`
`602`	`602`	`}`
`603`	`603`
`604`		`- public static string SafeSubstring(this string value, int length)`
	`604`	`+ public static string SafeSubstring(this string value, int startIndex)`
`605`	`605`	`{`
`606`		`- return String.IsNullOrEmpty(value)`
`607`		`- ? String.Empty`
`608`		`- : value.Substring(Math.Min(length, value.Length));`
	`606`	`+ return SafeSubstring(value, startIndex, value.Length);`
`609`	`607`	`}`
`610`	`608`
`611`	`609`	`public static string SafeSubstring(this string value, int startIndex, int length)`