On Mon, 26 May 2025 16:37:01 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:
> SonarCloud complains `ST_NEW` constant is not used. Looks to me the > constructor implicitly relies on default value for `state`. It would be > cleaner to initialize it explicitly > > Additional testing: > - [x] Linux x86_64 server fastdebug, `java/util/concurrent` src/java.base/share/classes/java/util/concurrent/StructuredTaskScopeImpl.java line 70: > 68: this.joiner = joiner; > 69: this.threadFactory = threadFactory; > 70: this.flock = ThreadFlock.open((name != null) ? name : > Objects.toIdentityString(this)); Unnecessary but okay. Do you mind moving it to the end of constructor as the fields are initialized in order declaration order in this constructor. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25452#discussion_r2107647128