Changeset 12050

Timestamp:

2006-07-07T09:46:40+12:00 (18 years ago)

Author:

mdewsnip

Message:

(FindBugs) Changed stripNL() to use a StringBuffer instead of repeatedly concatenating onto a String.

File:

• trunk/gli/src/org/greenstone/gatherer/util/Utility.java (modified) (1 diff)

trunk/gli/src/org/greenstone/gatherer/util/Utility.java

@@ -386 +386 @@
     static public String stripNL(String raw_string)
     {
-    String stripped_string = new String();
+    StringBuffer stripped_string_buffer = new StringBuffer();
     for (int i = 0; i < raw_string.length(); i++) {
         char raw_character = raw_string.charAt(i);
         if (raw_character != '\n' && raw_character != '\t') {
-        stripped_string = stripped_string + raw_character;
-        }
-    }
-    return stripped_string;
+        stripped_string_buffer.append(raw_character);
+        }
+    }
+    return stripped_string_buffer.toString();
     }
 }