requested changes

Author: an-dr-eas-k

src/observers/OmnisharpDebugModeLoggerObserver.ts

@@ -62,7 +62,7 @@ export class OmnisharpDebugModeLoggerObserver extends BaseLoggerObserver {
     }
 
     private handleOmnisharpProcessRequestStart(event: OmnisharpServerProcessRequestStart) {
-        this.logger.appendLine(`Processing ${event.name} queue, empty slots ${event.slots}`);
+        this.logger.appendLine(`Processing ${event.name} queue, available slots ${event.availableRequestSlots}`);
         this.logger.increaseIndent();
     }
 

src/omnisharp/EventType.ts

@@ -83,7 +83,7 @@ export enum EventType {
     DotNetTestRunInContextStart = 76,
     DotNetTestDebugInContextStart = 77,
     TelemetryErrorEvent = 78,
-    OmnisharpServerRequestCancelled = 79
+    OmnisharpServerRequestCancelled = 79,
 }
 
 //Note that the EventType protocol is shared with Razor.VSCode and the numbers here should not be altered

src/omnisharp/loggingEvents.ts

@@ -130,7 +130,7 @@ export class OmnisharpServerRequestCancelled implements BaseEvent {
 
 export class OmnisharpServerProcessRequestStart implements BaseEvent {
     type = EventType.OmnisharpServerProcessRequestStart;
-    constructor(public name: string, public slots: number) { }
+    constructor(public name: string, public availableRequestSlots: number) { }
 }
 
 export class OmnisharpEventPacketReceived implements BaseEvent {

src/omnisharp/prioritization.ts

@@ -15,7 +15,6 @@ const priorityCommands = [
 const normalCommands = [
     protocol.Requests.Completion,
     protocol.Requests.CompletionResolve,
-    protocol.Requests.AutoComplete,
     protocol.Requests.FilesChanged,
     protocol.Requests.FindSymbols,
     protocol.Requests.FindUsages,

src/omnisharp/protocol.ts

@@ -8,7 +8,6 @@ import { CompletionTriggerKind, CompletionItemKind, CompletionItemTag, InsertTex
 
 export module Requests {
     export const AddToProject = '/addtoproject';
-    export const AutoComplete = '/autocomplete';
     export const CodeCheck = '/codecheck';
     export const CodeFormat = '/codeformat';
     export const ChangeBuffer = '/changebuffer';

src/omnisharp/requestQueue.ts

@@ -88,10 +88,10 @@ class RequestQueue {
             return;
         }
 
-        const slots = this._maxSize - this._waiting.size;
-        this.eventStream.post(new OmnisharpServerProcessRequestStart(this._name, slots));
+        const availableRequestSlots = this._maxSize - this._waiting.size;
+        this.eventStream.post(new OmnisharpServerProcessRequestStart(this._name, availableRequestSlots));
 
-        for (let i = 0; i < slots && this._pending.length > 0; i++) {
+        for (let i = 0; i < availableRequestSlots && this._pending.length > 0; i++) {
             const item = this._pending.shift();
             item.startTime = Date.now();
 

test/unitTests/logging/OmnisharpDebugModeLoggerObserver.test.ts

@@ -50,7 +50,7 @@ suite("OmnisharpDebugModeLoggerObserver", () => {
         let event = new OmnisharpServerProcessRequestStart("foobar", 2);
         observer.post(event);
         expect(logOutput).to.contain(event.name);
-        expect(logOutput).to.contain(event.slots);
+        expect(logOutput).to.contain(event.availableRequestSlots);
     });
 
     test(`OmnisharpServerRequestCancelled: Name and Id is logged`, () => {