On Mon, 26 May 2025 17:17:30 GMT, Aleksey Shipilev 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 se
> 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`
Aleksey Shipilev has updat
On Mon, 26 May 2025 17:17:30 GMT, Aleksey Shipilev 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 se
On Mon, 26 May 2025 16:46:49 GMT, Alan Bateman wrote:
>> Aleksey Shipilev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Move to the end
>
> src/java.base/share/classes/java/util/concurrent/StructuredTaskScopeImpl.java
> line 70:
>
>>
On Mon, 26 May 2025 16:37:01 GMT, Aleksey Shipilev 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 f
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`
-
Commit messages:
- Fix