Author: rdonkin
Date: Thu Jul 23 10:37:36 2009
New Revision: 797010
URL: http://svn.apache.org/viewvc?rev=797010&view=rev
Log:
JSIEVE-53 Switch from StringBuffer to StringBuilder to improve performance
https://issues.apache.org/jira/browse/JSIEVE-53
Modified:
james/jsieve/trunk/main/src/main/jjtree/sieve/sieve.jjt
Modified: james/jsieve/trunk/main/src/main/jjtree/sieve/sieve.jjt
URL:
http://svn.apache.org/viewvc/james/jsieve/trunk/main/src/main/jjtree/sieve/sieve.jjt?rev=797010&r1=797009&r2=797010&view=diff
==============================================================================
--- james/jsieve/trunk/main/src/main/jjtree/sieve/sieve.jjt (original)
+++ james/jsieve/trunk/main/src/main/jjtree/sieve/sieve.jjt Thu Jul 23 10:37:36
2009
@@ -255,26 +255,25 @@
{
(quoted_string = <QUOTED_STRING> | multi_line = <MULTI_LINE>)
{
- final String stringValue;
+ final StringBuilder builder;
if (null != quoted_string)
- stringValue = quoted_string.image;
+ builder=new StringBuilder(quoted_string.image);
else if (null != multi_line)
- stringValue=multi_line.image;
+ builder=new StringBuilder(multi_line.image);
else
- stringValue=null;
- if (stringValue != null) {
+ builder=null;
+ if (builder != null) {
// Unescape and unquote
- final StringBuffer buffer = new StringBuffer(stringValue);
- buffer.deleteCharAt(stringValue.length() - 1);
- buffer.deleteCharAt(0);
+ builder.deleteCharAt(builder.length() - 1);
+ builder.deleteCharAt(0);
int i = 0;
- while (i < buffer.length()) {
- if ('\\' == buffer.charAt(i)) {
- buffer.deleteCharAt(i);
+ while (i < builder.length()) {
+ if ('\\' == builder.charAt(i)) {
+ builder.deleteCharAt(i);
}
i++;
}
- jjtThis.setValue(buffer.toString());
+ jjtThis.setValue(builder.toString());
}
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
