Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Apply changes directly to text buffer (#62337)" #62589

Merged
merged 1 commit into from
Jul 13, 2022
Merged

Conversation

tmat
Copy link
Member

@tmat tmat commented Jul 13, 2022

@tmat tmat requested a review from a team as a code owner July 13, 2022 01:27
@tmat tmat merged commit 0a39dcf into dotnet:main Jul 13, 2022
@ghost ghost added this to the Next milestone Jul 13, 2022
@tmat tmat deleted the Rev2 branch July 13, 2022 12:11
@CyrusNajmabadi
Copy link
Member

What was the issue here?

@genlu
Copy link
Member

genlu commented Jul 13, 2022

@tmat mentioned there is a post-processing step in ContainedDocument that cleans the buffers up after any changes to the buffer.

@tmat
Copy link
Member Author

tmat commented Jul 13, 2022

Working on a fix.

tmat added a commit to tmat/roslyn that referenced this pull request Jul 13, 2022
tmat added a commit to tmat/roslyn that referenced this pull request Jul 13, 2022
tmat added a commit to tmat/roslyn that referenced this pull request Jul 14, 2022
tmat added a commit to tmat/roslyn that referenced this pull request Jul 14, 2022
tmat added a commit that referenced this pull request Jul 18, 2022
* Revert "Revert "Apply changes directly to text buffer (#62337)" (#62589)"

This reverts commit 0a39dcf.

* Apply changes to ContainedDocument subject buffer
adamperlin pushed a commit to adamperlin/roslyn that referenced this pull request Jul 19, 2022
* Revert "Revert "Apply changes directly to text buffer (dotnet#62337)" (dotnet#62589)"

This reverts commit 0a39dcf.

* Apply changes to ContainedDocument subject buffer
@allisonchou allisonchou modified the milestones: Next, 17.4 P1 Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants