RFR (XS) 8076759: AbstractStringBuilder.append(...) should ensure enough capacity for the upcoming "trivial" append calls
aleksey.shipilev at oracle.com
Tue May 5 17:47:44 UTC 2015
Here is a new webrev:
Pruned the implementation details from expandCapacity Javadoc, and about
to submit a CCC for it.
On 05.05.2015 16:33, Roger Riggs wrote:
> Hi Aleksey,
> Thanks for checking the rounding alternative.
> As for the CCC, since the implementation details are in the javadoc
> then it will be needed either to remove the details or to update them.
> I'd be inclined to try to remove them since they are there primarily for
> Thanks, Roger
> On 5/5/2015 4:31 AM, Aleksey Shipilev wrote:
>> Hi Roger,
>> On 05/01/2015 08:19 PM, Roger Riggs wrote:
>>> Is there any additional benefit by rounding up the next multiple of 4
>>> or 8.
>>> That would avoid a few wasted bytes at the end of the buffer modulo the
>>> allocation size.
>> It does not seem to help any further. Tried "plus32-round8", as in:
>> ...and it performs similar to "plus32":
>>> Otherwise, looks fine to me also.
>> I actually wonder if my change in ensureCapacity Javadoc requires a CCC?
>> On that topic, I also tempted to remove the implementation details from
>> the Javadoc there, since it does not play well with "describe what you
>> will do, not how would you do it".
More information about the core-libs-dev