[
https://issues.apache.org/jira/browse/WICKET-7047?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17712890#comment-17712890
]
ASF GitHub Bot commented on WICKET-7047:
----------------------------------------
theigl commented on code in PR #579:
URL: https://github.com/apache/wicket/pull/579#discussion_r1168181660
##########
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:
We have the exact same logic in `Strings.escapeMarkup`:
https://github.com/apache/wicket/blob/d51883862a448e26f8d973a24756ea7e03259de7/wicket-util/src/main/java/org/apache/wicket/util/string/Strings.java#L305
A constant would have to be something like
`ESTIMATED_OVERHEAD_OF_MULTILINE_MARKUP`. I think a comment would be better.
> 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)
