XenoAmess commented on pull request #784: URL: https://github.com/apache/commons-lang/pull/784#issuecomment-933970122
> @XenoAmess Could you please be more descriptive? The only `String.valueOf` I have added are in high level methods, main logic does not. Actually I removed `String.valueOf` from loops. Sorry for answering in comment, but there's no way to add reply to your comment. @HubertWo Hi. I find some time today for this question (finally). `The only `String.valueOf` I have added are in high level methods, main logic does not` This statement be correct, but also be where I worried about performance. Please look at the sources in JDK, where `java.lang.AbstractStringBuilder#append(boolean b)` In short, for basic types, should not convert them to String, but use StringBuilder directly, will bring far better performance. BUT I just foundout that you did not bring this issue, this issue is already in commons-lang before your pr. So I will not stop you from merging this in, but still I will find time to refactor the whole `join` serial functions later. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
