RFR: 8294954: Remove superfluous ResourceMarks when using LogStream

David Holmes dholmes at openjdk.org
Fri Oct 7 13:21:21 UTC 2022


On Fri, 7 Oct 2022 11:19:55 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> Hi,
> 
> I went through all of the places where LogStreams are created and removed the unnecessary ResourceMarks. I also added a ResourceMark in one place, where it was needed because of a call to `::name_and_sig_as_C_string` and moved one to the smallest scope where it is used.

How are you defining "unnecessary"? Are these unnecessary because there is zero resource allocation involved? Or "unnecessary" because a ResourceMark higher up the call stack covers it?

-------------

PR: https://git.openjdk.org/jdk/pull/10602


More information about the hotspot-dev mailing list