DS-48860: Update warnings

Updated some SDK code to avoid warning messages from code scanning
tools. This includes a change to the StringReader#skip method, which now
explicitly checks for negative values.

scim2-sdk-common/src/main/java/com/unboundid/scim2/common/utils/Parser.java

@@ -50,6 +50,7 @@ private static final class StringReader extends Reader
   {
     @NotNull
     private final String string;
+
     private int pos;
     private int mark;
 
@@ -122,12 +123,23 @@ public void reset()
 
     /**
      * {@inheritDoc}
+     *
+     * @throws IllegalArgumentException   If {@code n} is negative.
      */
     @Override
-    public long skip(final long n)
+    public long skip(final long n) throws IllegalArgumentException
     {
+      if (n < 0L)
+      {
+        throw new IllegalArgumentException(
+            "Attempted to skip a negative number of characters.");
+      }
       long chars = Math.min(string.length() - pos, n);
-      pos += chars;
+
+      // The left side of Math.min() is an integer, and 'chars' will always be
+      // non-negative, so 'chars' can always be represented as an int.
+      pos += (int) chars;
+
       return chars;
     }
 

scim2-sdk-common/src/main/java/com/unboundid/scim2/common/utils/StaticUtils.java

@@ -28,16 +28,12 @@
 import java.util.List;
 import java.util.Objects;
 import java.util.Set;
-import java.util.regex.Pattern;
 
 /**
  * This class provides a number of static utility functions.
  */
 public final class StaticUtils
 {
-  @NotNull
-  private static final Pattern SEPARATOR = Pattern.compile("\\s*,\\s*");
-
   /**
    * Prevent this class from being instantiated.
    */
@@ -240,7 +236,13 @@ public static <T> Set<T> arrayToSet(@NotNull final T... i)
   @NotNull
   public static String[] splitCommaSeparatedString(@NotNull final String str)
   {
-    return SEPARATOR.split(str.trim());
+    String[] separatedArray = str.split(",");
+    for (int i = 0; i < separatedArray.length; i++)
+    {
+      separatedArray[i] = separatedArray[i].trim();
+    }
+
+    return separatedArray;
   }