[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-runtime-dev
Subject:    Integrated: 8322747: StringTable should be AllStatic
From:       Kim Barrett <kbarrett () openjdk ! org>
Date:       2023-12-31 17:29:45
Message-ID: 2RwZILq4ivVxI1_bQaLneu-Lar0iKguS7DM5SagFLl0=.5ac960e6-f960-41cc-accf-76d6979bd981 () github ! com
[Download RAW message or body]

On Wed, 27 Dec 2023 03:52:49 GMT, Kim Barrett <kbarrett@openjdk.org> wrote:

> Please review this trivial change to StringTable, making it derive from
> AllStatic rather than CHeapObj<mtSymbol>, to match the implementation.
> 
> While I was there, also did a little cleanup of related forward and friend
> declarations.  StringTableCreateEntry doesn't exist.  Don't need forward
> declaration for StringTable right before it's definition.  StringTable doesn't
> need to befriend Symbol.
> 
> Testing: mach5 tier1

This pull request has now been integrated.

Changeset: 518ec971
Author:    Kim Barrett <kbarrett@openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/518ec9711411e6825668f72503a2e96824cd37ba
Stats:     6 lines in 1 file changed: 0 ins; 4 del; 2 mod

8322747: StringTable should be AllStatic

Reviewed-by: eosterlund

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

PR: https://git.openjdk.org/jdk/pull/17193
[prev in list] [next in list] [prev in thread] [next in thread]