Skip to content

Ensure that tokens are re-fetched to the position in the reset point in SyntaxParser.Reset#48393

Merged
jaredpar merged 3 commits intodotnet:release/dev16.8from
AlekseyTs:WorkItem1179569_01
Oct 9, 2020
Merged

Ensure that tokens are re-fetched to the position in the reset point in SyntaxParser.Reset#48393
jaredpar merged 3 commits intodotnet:release/dev16.8from
AlekseyTs:WorkItem1179569_01

Conversation

@AlekseyTs
Copy link
Contributor

@AlekseyTs AlekseyTs commented Oct 7, 2020

Fixes AB#1179569.

Feedback ticket ID is 1224085

@AlekseyTs AlekseyTs added PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. Area-Compilers Tenet-Reliability Customer telemetry indicates that the product is failing in a crash/hang/dataloss manner. and removed PR For Personal Review Only The PR doesn’t require anyone other than the developer to review it. labels Oct 7, 2020
@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-compiler Please review a fix for a Watson bug.

@AlekseyTs AlekseyTs marked this pull request as ready for review October 7, 2020 17:25
@AlekseyTs AlekseyTs requested review from a team as code owners October 7, 2020 17:25
Copy link
Member

Choose a reason for hiding this comment

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

nit: consider adding a similar assertion in GetResetPoint

Copy link
Member

@jcouv jcouv left a comment

Choose a reason for hiding this comment

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

LGTM Thanks (iteration 3)

@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-compiler Please review a fix for a Watson bug, need a second sign-off.

@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-ide, @dotnet/roslyn-infrastructure Are the failures of integration tests expected?

@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-compiler Please review a fix for a Watson bug, need a second sign-off.

@AlekseyTs
Copy link
Contributor Author

@cston Please review.

@jaredpar jaredpar merged commit 6ad4b00 into dotnet:release/dev16.8 Oct 9, 2020
@jcouv jcouv added this to the 16.8.P5 milestone Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved to merge Area-Compilers Tenet-Reliability Customer telemetry indicates that the product is failing in a crash/hang/dataloss manner.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants