Skip to content

Fix System.Thread namespace to System.Threading.Thread in Memory-model.md#127969

Open
Copilot wants to merge 1 commit intomainfrom
copilot/fix-system-thread-namespace
Open

Fix System.Thread namespace to System.Threading.Thread in Memory-model.md#127969
Copilot wants to merge 1 commit intomainfrom
copilot/fix-system-thread-namespace

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 8, 2026

Fixes incorrect namespace references in docs/design/specs/Memory-model.md.

System.Thread.VolatileRead, System.Thread.VolatileWrite, and System.Thread.MemoryBarrier are corrected to System.Threading.Thread.VolatileRead, System.Threading.Thread.VolatileWrite, and System.Threading.Thread.MemoryBarrier.

Note

This PR was created by GitHub Copilot.

Copilot AI requested review from Copilot and removed request for Copilot May 8, 2026 19:35
Copilot finished work on behalf of jkotas May 8, 2026 19:36
Copilot AI requested a review from jkotas May 8, 2026 19:36
@jkotas jkotas requested a review from VSadov May 8, 2026 19:53
@jkotas jkotas marked this pull request as ready for review May 8, 2026 19:53
Copilot AI review requested due to automatic review settings May 8, 2026 19:53
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
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

Updates the .NET memory model design spec to correct fully-qualified API references for Thread-based fence/volatile helpers, aligning the document with the actual System.Threading.Thread type.

Changes:

  • Replace System.Thread.VolatileRead with System.Threading.Thread.VolatileRead.
  • Replace System.Thread.VolatileWrite with System.Threading.Thread.VolatileWrite.
  • Replace System.Thread.MemoryBarrier with System.Threading.Thread.MemoryBarrier.

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