-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Replace Interlocked.{Compare}Exchange Int32 overloads with generic <bool> variants #9480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace Interlocked.{Compare}Exchange Int32 overloads with generic <bool> variants #9480
Conversation
The PR looks good to me. @h3xds1nz if you could resolve the merge conflicts please? |
f32b763
to
9a39289
Compare
@siagupta0202 Done, should be fine :) |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9480 +/- ##
===================================================
- Coverage 10.99508% 10.94612% -0.04896%
===================================================
Files 3215 3215
Lines 648472 648440 -32
Branches 71534 71527 -7
===================================================
- Hits 71300 70979 -321
- Misses 576170 576469 +299
+ Partials 1002 992 -10
Flags with carried forward coverage won't be shown. Click here to find out more. |
@h3xds1nz Thank you so much for all your contributions! |
@siagupta0202 Thank you for reviewing :) |
Description
As dotnet/runtime - #65184 was approved last week and changes were made in the runtime repo to unconstrain the generic overload from class-only behaviour for
Interlocked.Exchange
andInterlocked.CompareExchange
, we can adjust the code in the WPF repo for improved code quality as well.Given how few places we rely on
Interlocked
functionality, this should be easy to review.Customer Impact
None, just internal code quality changes that do not measurably affect performance in any direction.
Regression
No.
Testing
Local build/CI.
Risk
None.
Microsoft Reviewers: Open in CodeFlow