[ https://issues.apache.org/jira/browse/WICKET-7047?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17712893#comment-17712893 ]
ASF GitHub Bot commented on WICKET-7047: ---------------------------------------- theigl commented on code in PR #579: URL: https://github.com/apache/wicket/pull/579#discussion_r1168193350 ########## wicket-util/src/main/java/org/apache/wicket/util/string/Strings.java: ########## @@ -1089,7 +1089,7 @@ public static CharSequence toMultilineMarkup(final CharSequence s) return null; } - final AppendingStringBuffer buffer = new AppendingStringBuffer(); + final AppendingStringBuffer buffer = new AppendingStringBuffer((int) (s.length() * 1.1)); Review Comment: I don't think so. The additional 10% should be enough to accommodate markup and escaping overhead in most cases. And if it isn't enough, the buffer is resized once. This is *much* better than resizing the buffer almost every time as it is now. Imho there is no need to configure or optimize this further. > Improve initial buffer capacity for Strings.toMultilineMarkup > ------------------------------------------------------------- > > Key: WICKET-7047 > URL: https://issues.apache.org/jira/browse/WICKET-7047 > Project: Wicket > Issue Type: Improvement > Components: wicket-core > Affects Versions: 9.12.0 > Reporter: Thomas Heigl > Assignee: Thomas Heigl > Priority: Major > > We currently create an AppendingStringBuffer with default capacity of 16 in > Strings#toMultilineMarkup. Since we know the size of the original string and > can take a guess at the additional length required for markup, it makes sense > to size this buffer appropriately. -- This message was sent by Atlassian Jira (v8.20.10#820010)