RFR(s): various stringStream issues
Thomas Stüfe
thomas.stuefe at gmail.com
Thu Jun 27 11:54:17 UTC 2019
Hi all.
this is a smallish change which rewrites stringStream to achieve two goals:
- have a buffer size cap in dynamic mode: faulty - or just very chatty -
printing code can use up lots of memory, and we have no inbuilt limit. We
should have one. This is similar to
https://bugs.openjdk.java.net/browse/JDK-8220394
- Since 8224193, stringStream uses no resource area anymore as backing
buffer but C-heap. To reduce the number of malloc calls for small instances
of this stream I added a small internal buffer to be used for small output.
I rewrote stringStream to be more in line with how the rest of our code
looks today (constness, initializer lists etc).
I also tweaked the tests a lot and believe this code pretty well tested.
JBS: https://bugs.openjdk.java.net/browse/JDK-8224213
webrev:
http://cr.openjdk.java.net/~stuefe/webrevs/8224213-stringstream-maxcap/webrev.00/webrev/index.html
Thank you, Thomas
More information about the hotspot-runtime-dev
mailing list