Re: RFR: 8291641: Optimize StackTraceElement.toString() [v7]

2022-08-06 Thread David Schlosnagle
> I would like to contribute an optimized version of > `StackTraceElement#toString()` that uses a single StringBuilder throughout > creation to avoid intermediate `String` allocations. > `StackTraceElement#toString()` is used in a number of JDK code paths > including `Throwable#printStackTrace(

Re: RFR: 8291641: Optimize StackTraceElement.toString() [v7]

2022-08-08 Thread Max Mielchen
On Sat, 6 Aug 2022 14:55:17 GMT, David Schlosnagle wrote: >> I would like to contribute an optimized version of >> `StackTraceElement#toString()` that uses a single StringBuilder throughout >> creation to avoid intermediate `String` allocations. >> `StackTraceElement#toString()` is used in a n

Re: RFR: 8291641: Optimize StackTraceElement.toString() [v7]

2022-08-09 Thread Mandy Chung
On Sat, 6 Aug 2022 14:55:17 GMT, David Schlosnagle wrote: >> I would like to contribute an optimized version of >> `StackTraceElement#toString()` that uses a single StringBuilder throughout >> creation to avoid intermediate `String` allocations. >> `StackTraceElement#toString()` is used in a n

Re: RFR: 8291641: Optimize StackTraceElement.toString() [v7]

2022-08-09 Thread Max Mielchen
On Sat, 6 Aug 2022 14:55:17 GMT, David Schlosnagle wrote: >> I would like to contribute an optimized version of >> `StackTraceElement#toString()` that uses a single StringBuilder throughout >> creation to avoid intermediate `String` allocations. >> `StackTraceElement#toString()` is used in a n

Re: RFR: 8291641: Optimize StackTraceElement.toString() [v7]

2022-08-09 Thread David Schlosnagle
On Tue, 9 Aug 2022 18:35:05 GMT, Mandy Chung wrote: >> David Schlosnagle has updated the pull request with a new target base due to >> a merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 11 additional >> commit

Re: RFR: 8291641: Optimize StackTraceElement.toString() [v7]

2022-08-09 Thread David Schlosnagle
On Mon, 8 Aug 2022 18:55:39 GMT, Max Mielchen wrote: >> David Schlosnagle has updated the pull request with a new target base due to >> a merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains 11 additional >> commi