Skip to content

Fix null annotation of SyntaxFactory.UsingStatement - #51984

Merged
jcouv merged 3 commits into
dotnet:mainfrom
Youssef1313:patch-4
Mar 23, 2021
Merged

jcouv merged 3 commits into
dotnet:mainfrom
Youssef1313:patch-4

Conversation

@Youssef1313

Copy link
Copy Markdown
Member

Fixes #51924

@Youssef1313
Youssef1313 requested a review from a team as a code owner March 19, 2021 10:38
@ghost ghost added the Area-Compilers label Mar 19, 2021
@jaredpar jaredpar added the Community The pull request was submitted by a contributor who is not a Microsoft employee. label Mar 19, 2021
public partial class SyntaxFactory
{
public static UsingStatementSyntax UsingStatement(SyntaxToken usingKeyword, SyntaxToken openParenToken, VariableDeclarationSyntax declaration, ExpressionSyntax expression, SyntaxToken closeParenToken, StatementSyntax statement)
public static UsingStatementSyntax UsingStatement(SyntaxToken usingKeyword, SyntaxToken openParenToken, VariableDeclarationSyntax? declaration, ExpressionSyntax? expression, SyntaxToken closeParenToken, StatementSyntax statement)

@RikkiGibson RikkiGibson Mar 19, 2021 •

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.

Should similar annotations be added to UsingStatementSyntax.Update methods above? #Resolved

@333fred

333fred commented Mar 23, 2021

Copy link
Copy Markdown
Member

@Youssef1313 looks like there's a conflict here.

@jcouv

jcouv commented Mar 23, 2021

Copy link
Copy Markdown
Contributor

Can be merged/squashed once CI is green. I just restarted some integration legs.

@jcouv
jcouv merged commit 58e6128 into dotnet:main Mar 23, 2021
@ghost ghost added this to the Next milestone Mar 23, 2021
@jcouv

jcouv commented Mar 23, 2021

Copy link
Copy Markdown
Contributor

Merged/squashed. Thanks @Youssef1313 !

@Youssef1313
Youssef1313 deleted the patch-4 branch March 24, 2021 09:44
@333fred 333fred mentioned this pull request Mar 25, 2021
@allisonchou allisonchou modified the milestones: Next, 16.10.P2 Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Community The pull request was submitted by a contributor who is not a Microsoft employee.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SyntaxFactory.UsingStatement incorrectly annotates 'declaration' and 'expression' parameters as disallowing null

7 participants