RFR: 8344461: Additional cleanup in NewThreadAction [v2]

[Stuart Marks]

On Wed, 4 Dec 2024 19:48:03 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Stuart Marks has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Make thread group fields final.
>
> src/java.rmi/share/classes/sun/rmi/runtime/RuntimeUtil.java line 60:
> 
>> 58:      * Cached reference to the system (root) thread group.
>> 59:      */
>> 60:     static final ThreadGroup systemThreadGroup;
> 
> Can this be private?  Also userThreadGroup below?

Good point. They weren't private in the original, which I just copied over, but I guess we're being a bit more meticulous about making things private these days.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22557#discussion_r1870368408