Skip to content

Fix issue where source files could get corrupted.#2167

Merged
CyrusNajmabadi merged 5 commits into
masterfrom
documentRegistry
Feb 28, 2015
Merged

Fix issue where source files could get corrupted.#2167
CyrusNajmabadi merged 5 commits into
masterfrom
documentRegistry

Conversation

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

This could happen when you had multiple language services, and they were sharing some files.
If a file got edited in one LS, it could get corrupted in the other. Now, the DocumentRegistry
serves as the canonical source of 'good' source files. Language services always go to it to get
the correct source file instead of trying to manually update their own source files when they
are notified about changes from the host.

This could happen when you had multiple language services, and they were sharing some files.
If a file got edited in one LS, it could get corrupted in the other.  Now, the DocumentRegistry
serves as the canonical source of 'good' source files.  Language services always go to it to get
the correct source file instead of trying to manually update their own source files when they
are notified about changes from the host.
@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Feb 28, 2015

unit test?

👍

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor Author

@mhegazy adding tests.

CyrusNajmabadi added a commit that referenced this pull request Feb 28, 2015
Fix issue where source files could get corrupted.
@CyrusNajmabadi CyrusNajmabadi merged commit 419d379 into master Feb 28, 2015
@CyrusNajmabadi CyrusNajmabadi deleted the documentRegistry branch February 28, 2015 00:29
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants