[ 
https://issues.apache.org/jira/browse/HUPA-58?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Manuel Carrasco Moñino resolved HUPA-58.
----------------------------------------

    Resolution: Fixed

Partially applied in trunk
I've applied your patch only to server classes, after fixing some lines which 
break tests.
In shared classes, code is going to be compiled into javascript, and I think 
the patch doesnt worth it because the '+' operator in javascript works 
different than in java, and '+' is easier to handle by gwt compiler.

> Strings concatenated in loops
> -----------------------------
>
>                 Key: HUPA-58
>                 URL: https://issues.apache.org/jira/browse/HUPA-58
>             Project: JAMES Hupa
>          Issue Type: Improvement
>            Reporter: Eric Jablow
>            Assignee: Manuel Carrasco Moñino
>            Priority: Minor
>         Attachments: concatenation-in-loop.patch
>
>   Original Estimate: 0.17h
>  Remaining Estimate: 0.17h
>
> A few Java files have code like
> String ret = "";
> for (String s : ...) {
>   ret += s +" ";
> }
> StringBuilder.append() is faster and does not allocate new objects.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to