Integrated: 8294954: Remove superfluous ResourceMarks when using LogStream

[Casper Norrbin]

On Fri, 21 Mar 2025 15:38:21 GMT, Casper Norrbin <cnorrbin at openjdk.org> wrote:

> Hi everyone,
> 
> This PR removes redundant `ResourceMark` instances where `LogStream` is used. Previously, `LogStream` inherited from `ResourceObj`, which required a `ResourceMark`, but this is no longer the case, making these instances unnecessary.
> 
> Process:
> 1. I added assertions to check for resource unwinding in places where `ResourceMark`s were used with `LogStream`s.
> 2. Ran tests up to tier7 to confirm no unwinding was happening. This helped filter out cases where `ResourceMark`s were still required for other reasons.
> 3. Manually verified the remaining cases by tracing function calls to ensure the `ResourceMark`s were truly unnecessary.
> 4. Removed the redundant `ResourceMark` instances.

This pull request has now been integrated.

Changeset: 89e5c42d
Author:    Casper Norrbin <cnorrbin at openjdk.org>
Committer: Johan Sjölen <jsjolen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/89e5c42d909344d75266a203d7e6b6bb1ad4aea6
Stats:     40 lines in 26 files changed: 1 ins; 39 del; 0 mod

8294954: Remove superfluous ResourceMarks when using LogStream

Reviewed-by: dholmes, jsjolen

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

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