Skip to content

Apply MemberNotNull#48570

Merged
333fred merged 1 commit intodotnet:masterfrom
333fred:membernotnull
Oct 15, 2020
Merged

Apply MemberNotNull#48570
333fred merged 1 commit intodotnet:masterfrom
333fred:membernotnull

Conversation

@333fred
Copy link
Copy Markdown
Member

@333fred 333fred commented Oct 13, 2020

Fixes #41964.

@333fred 333fred requested a review from a team as a code owner October 13, 2020 21:07
@Dotnet-GitSync-Bot
Copy link
Copy Markdown
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@333fred
Copy link
Copy Markdown
Member Author

333fred commented Oct 13, 2020

@dotnet/roslyn-compiler for review.

if (!hasErrors)
{
Debug.Assert(AttributeClass is not null);
Debug.Assert(AttributeConstructor is not null);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is object is shorter :trollface:

@jcouv jcouv added the Concept-Null Annotations The issue involves annotating an API for nullable reference types label Oct 14, 2020
@333fred 333fred merged commit 857673e into dotnet:master Oct 15, 2020
@333fred 333fred deleted the membernotnull branch October 15, 2020 00:38
@ghost ghost added this to the Next milestone Oct 15, 2020
@allisonchou allisonchou modified the milestones: Next, 16.9.P2 Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Concept-Null Annotations The issue involves annotating an API for nullable reference types

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use MemberNotNull when available

6 participants