Skip to content

Fix null annotation of SyntaxFactory.UsingStatement#51984

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

Fix null annotation of SyntaxFactory.UsingStatement#51984
jcouv merged 3 commits intodotnet:mainfrom
Youssef1313:patch-4

Conversation

@Youssef1313
Copy link
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)
Copy link
Member

@RikkiGibson RikkiGibson Mar 19, 2021

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
Copy link
Member

333fred commented Mar 23, 2021

@Youssef1313 looks like there's a conflict here.

@jcouv
Copy link
Member

jcouv commented Mar 23, 2021

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
Copy link
Member

jcouv commented Mar 23, 2021

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