Skip to content

renames to IsMultithreadingSupported#124653

Merged
pavelsavara merged 8 commits intodotnet:mainfrom
pavelsavara:st_prep
Feb 20, 2026
Merged

renames to IsMultithreadingSupported#124653
pavelsavara merged 8 commits intodotnet:mainfrom
pavelsavara:st_prep

Conversation

@pavelsavara
Copy link
Member

Boring prep for #124603

@pavelsavara pavelsavara added this to the 11.0.0 milestone Feb 20, 2026
@pavelsavara pavelsavara self-assigned this Feb 20, 2026
@pavelsavara pavelsavara requested a review from lewing as a code owner February 20, 2026 16:00
Copilot AI review requested due to automatic review settings February 20, 2026 16:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR performs a comprehensive rename to prepare for API changes in PR #124603. The changes rename internal and test-related properties/methods from "Threading" to "Multithreading" and from "SingleThreaded" to "MultithreadingSupported" (with inverted logic).

Changes:

  • Renamed PlatformDetection.IsThreadingSupportedIsMultithreadingSupported and IsThreadingNotSupportedIsNotMultithreadingSupported across all test files
  • Renamed internal Thread.IsSingleThreadedThread.IsMultithreadingSupported and Thread.ThrowIfSingleThreaded()Thread.ThrowIfMultithreadingIsNotSupported() with proper logic inversions
  • Updated documentation and comments to reflect new terminology

Reviewed changes

Copilot reviewed 280 out of 281 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/tests/Common/CoreCLRTestLibrary/PlatformDetection.cs Renamed test utility properties for multithreading detection
src/libraries/Common/tests/TestUtilities/System/PlatformDetection.cs Renamed test utility properties for multithreading detection
src/coreclr/System.Private.CoreLib/src/System/Threading/Thread.CoreCLR.cs Updated SpinWait to check IsMultithreadingSupported instead of IsSingleThreaded with inverted logic
src/libraries/System.Threading.Tasks.Parallel/src/System/Threading/Tasks/Parallel.cs Renamed IsSingleThreaded → IsMultithreadingSupported with proper logic inversion
src/libraries/System.Threading.Tasks.Parallel/src/System/Threading/Tasks/TaskReplicator.cs Updated condition to use !IsMultithreadingSupported instead of IsSingleThreaded
src/libraries/System.Private.CoreLib/src/System/Threading/*.cs Updated threading infrastructure to use new naming with proper logic inversions
src/libraries/System.Private.CoreLib/src/System/IO/*.cs Updated Stream and TextWriter to use new naming with proper logic inversions
src/mono/wasm/threads.md Updated documentation to reference new API names
src/mono/System.Private.CoreLib/src/System/Threading/Monitor.Mono.cs Updated method call to ThrowIfMultithreadingIsNotSupported()
eng/testing/tests.browser.targets Updated comment to reference IsMultithreadingSupported
200+ test files Updated ConditionalFact/ConditionalTheory/ConditionalClass attributes and ActiveIssue comments

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @agocke, @VSadov
See info in area-owners.md if you want to be subscribed.

@pavelsavara
Copy link
Member Author

/ba-g untelated OOM and known issues

@pavelsavara pavelsavara merged commit 27cda88 into dotnet:main Feb 20, 2026
170 of 174 checks passed
@pavelsavara pavelsavara deleted the st_prep branch February 20, 2026 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants