-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Refactoring with .NET 5 Unsafe APIs #3510
Refactoring with .NET 5 Unsafe APIs #3510
Conversation
Thanks Sergio0694 for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌 |
Fixed merge conflicts and marked as ready for review, as .NET 5 RC1 is released with "go live" license! 🚀 |
Working around lack of annotations below .NET Standard 2.1, see dotnet/runtime#42827 for more info
Not needed yet as there are no new relevant APIs here
aeb015f
to
da5264d
Compare
The 5.0.0 packages are out on NuGet and I've updated this PR, we're good to go! 🚀🚀🚀 |
The CI failed with this error:
I've seen this on and off seemingly at random in a number of failed CI runs across multiple PRs of mine now. My guess is that that crash was caused by the Basically, whenever the CI happened to run in a server in some region with the decimal specified using EDIT: all is fine now! 🎉 |
Hello @michael-hawker! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
Close #3509
PR Type
What kind of change does this PR introduce?
What is the current behavior?
There are a number of usages of
Unsafe.AsRef<T>(null)
and similar that are verbose.What is the new behavior?
These occurrences are replaced using the new
Unsafe.NullRef<T>()
andUnsafe.IsNullRef<T>(...)
APIs.Notes
Leaving this as draft as the new
System.Runtime.CompilerServices.Unsafe
package is still in release candidate.PR Checklist
Please check if your PR fulfills the following requirements:
Pull Request has been submitted to the documentation repository instructions. Link:Sample in sample app has been added / updated (for bug fixes / features)Icon has been created (if new sample) following the Thumbnail Style Guide and templates