RFR: JDK-8304684: Memory leak in DirectivesParser::set_option_flag [v4]

Dean Long dlong at openjdk.org
Tue Mar 28 21:19:34 UTC 2023


On Tue, 28 Mar 2023 14:30:55 GMT, Justin King <jcking at openjdk.org> wrote:

>> Update `DirectivesSet` to take ownership of string options in some cases, to not leak memory.
>
> Justin King has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Adjust logic based on review
>   
>   Signed-off-by: Justin King <jcking at google.com>

Looks good.

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

Marked as reviewed by dlong (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13125#pullrequestreview-1361918952


More information about the hotspot-compiler-dev mailing list