CR feedback: use server.makeRequest<T>'s proper cancellation token handling

Author: christianscheuer

src/features/completionItemProvider.ts

@@ -41,9 +41,9 @@ export default class OmniSharpCompletionItemProvider extends AbstractSupport imp
         }
 
         try {
-            let responses = await serverUtils.autoComplete(this._server, req);
+            let responses = await serverUtils.autoComplete(this._server, req, token);
 
-            if (!responses || token && token.isCancellationRequested) {
+            if (!responses) {
                 return;
             }
 

src/omnisharp/utils.ts

@@ -7,8 +7,8 @@ import { OmniSharpServer } from './server';
 import * as protocol from './protocol';
 import * as vscode from 'vscode';
 
-export async function autoComplete(server: OmniSharpServer, request: protocol.AutoCompleteRequest) {
-    return server.makeRequest<protocol.AutoCompleteResponse[]>(protocol.Requests.AutoComplete, request);
+export async function autoComplete(server: OmniSharpServer, request: protocol.AutoCompleteRequest, token: vscode.CancellationToken) {
+    return server.makeRequest<protocol.AutoCompleteResponse[]>(protocol.Requests.AutoComplete, request, token);
 }
 
 export async function codeCheck(server: OmniSharpServer, request: protocol.Request, token: vscode.CancellationToken) {