ppkarwasz commented on PR #1332: URL: https://github.com/apache/commons-lang/pull/1332#issuecomment-2542081797
@Serwios, String concatenation is IMHO more readable than the equivalent `StringBuilder` construction, but changing this in a single method that is probably not used and certainly not tested, doesn't make sense to me. Could you go the extra mile of: - proposing on the `[email protected]` mailing list to replace simple `StringBuilder` constructs with string concatenation. Whether to use `StringBuilder`, string concatenation or `String.format()` is mostly a matter of taste and IDEs can refactor each one of these formats into another. - if the community accepts the change, apply it **everywhere**? -- 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]
