Skip to content

Use [MemberNotNull] to reduce use of = null!; - #37490

Merged
stephentoub merged 2 commits into
dotnet:masterfrom
stephentoub:membernotnull
Jun 9, 2020
Merged

Use [MemberNotNull] to reduce use of = null!;#37490
stephentoub merged 2 commits into
dotnet:masterfrom
stephentoub:membernotnull

Conversation

@stephentoub

@stephentoubstephentoub commented Jun 5, 2020

Copy link
Copy Markdown
Member

stephentoub
stephentoub commented Jun 5, 2020
View reviewed changes
Comment threadsrc/libraries/System.Drawing.Common/ref/System.Drawing.Common.cs Outdated
bartonjs
bartonjs reviewed Jun 5, 2020
View reviewed changes
Comment threadsrc/libraries/Common/src/System/Security/Cryptography/Asn1Reader/AsnWriter.cs Outdated
@stephentoub
stephentoubforce-pushed the membernotnull branch 4 times, most recently from 4df424b to 51aef9dCompareJune 8, 2020 01:23
safern
safern reviewed Jun 8, 2020
View reviewed changes
Comment threadsrc/libraries/System.Diagnostics.TraceSource/src/System/Diagnostics/SwitchLevelAttribute.cs Outdated
@stephentoub
stephentoubforce-pushed the membernotnull branch 3 times, most recently from a9ce012 to aa20fd7CompareJune 8, 2020 21:41
@stephentoubstephentoub reopened this Jun 9, 2020
@stephentoub
stephentoub merged commit 5f43e51 into dotnet:masterJun 9, 2020
@stephentoub
stephentoub deleted the membernotnull branch June 9, 2020 09:28
@ghostghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@tannergoodingtannergoodingtannergooding left review comments
@bartonjsbartonjsbartonjs approved these changes
@safernsafernsafern approved these changes

Assignees

No one assigned

Labels

area-Meta

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@stephentoub@tannergooding@bartonjs@safern@Dotnet-GitSync-Bot