Changeset 7087

Timestamp:

11/15/16 20:54:14 (8 years ago)

Author:

Oliver Schonefeld

Message:

• push pre-allocation of char buffer further up (now, only one buffer is allocated for each parse)

File:

• FCSSimpleEndpoint/trunk/src/main/java/eu/clarin/sru/server/fcs/parser/QueryParser.java (modified) (3 diffs)

FCSSimpleEndpoint/trunk/src/main/java/eu/clarin/sru/server/fcs/parser/QueryParser.java

@@ -197 +197 @@
             extends FCSParserBaseVisitor<Void> {
         private final Deque<Object> stack;
+        /*
+         * pre-allocate buffer to store chars returned by Character.toChars in
+         * unescapeString()
+         */
+        private final char[] buf = new char[2];
 
 
@@ -592 +597 @@
             /* process escape sequences, if present */
             if (regex.indexOf('\\') != -1) {
-                regex = unescapeString(regex);
+                regex = unescapeString(regex, buf);
             }
 
@@ -798 +803 @@
 
 
-    private static String unescapeString(String s) {
-        /*
-         * buffer to store chars by Character.toChars; pre-allocate to avoid
-         * allocation for every codepoint.
-         */
-        final char[] buf = new char[2];
-
+    private static String unescapeString(String s, char[] buf) {
         final StringBuilder sb = new StringBuilder();
         for (int i = 0; i < s.length(); i++) {