Skip to content

Commit f2ea93f

Browse files
authored
Merge pull request dotnet#4170 from JoeRobich/check-for-changes
Do not call updateBuffer if there are no changes.
2 parents 3e4dd35 + 9d9e5fa commit f2ea93f

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/features/changeForwarding.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,21 @@ function forwardDocumentChanges(server: OmniSharpServer): IDisposable {
1414

1515
return workspace.onDidChangeTextDocument(event => {
1616

17-
let { document } = event;
17+
let { document, contentChanges } = event;
1818
if (document.isUntitled || document.languageId !== 'csharp' || document.uri.scheme !== 'file') {
1919
return;
2020
}
2121

22+
if (contentChanges.length === 0) {
23+
// This callback fires with no changes when a document's state changes between "clean" and "dirty".
24+
return;
25+
}
26+
2227
if (!server.isRunning()) {
2328
return;
2429
}
2530

26-
const lineChanges = event.contentChanges.map(function (change): LinePositionSpanTextChange {
31+
const lineChanges = contentChanges.map(function (change): LinePositionSpanTextChange {
2732
const range = change.range;
2833
return {
2934
NewText: change.text,

0 commit comments

Comments
 (0)