added configurable option "useEditorFormattingSettings"

Author: filipw

package.json

@@ -365,6 +365,11 @@
           "type": "number",
           "default": 250,
           "description": "The maximum number of projects to be shown in the 'Select Project' dropdown (maximum 250)."
+        },
+        "omnisharp.useEditorFormattingSettings": {
+          "type": "boolean",
+          "default": true,
+          "description": "Specifes whether OmniSharp should use VS Code editor settings for C# code formatting (use of tabs, indentation size)."
         }
       }
     },

src/omnisharp/launcher.ts

@@ -161,13 +161,14 @@ function launch(cwd: string, args: string[]): Promise<LaunchResult> {
     return PlatformInformation.GetCurrent().then(platformInfo => {
         const options = Options.Read();
 
-        let editorConfig = vscode.workspace.getConfiguration('editor');
-
-        let envVars = {
-            'formattingOptions:useTabs': !editorConfig.get('insertSpaces', true),
-            'formattingOptions:tabSize': editorConfig.get('tabSize', 4),
-            'formattingOptions:indentationSize': editorConfig.get('tabSize', 4)
-        };
+        let envVars = {};
+        if (options.useEditorFormattingSettings) 
+        {
+            let editorConfig = vscode.workspace.getConfiguration('editor');
+            envVars['formattingOptions:useTabs'] = !editorConfig.get('insertSpaces', true);
+            envVars['formattingOptions:tabSize'] = editorConfig.get('tabSize', 4);
+            envVars['formattingOptions:indentationSize'] = editorConfig.get('tabSize', 4);
+        }
 
         if (options.path && options.useMono) {
             return launchNixMono(options.path, cwd, args, envVars);

src/omnisharp/options.ts

@@ -12,7 +12,8 @@ export class Options {
         public loggingLevel?: string,
         public autoStart?: boolean,
         public projectLoadTimeout?: number,
-        public maxProjectResults?: number) { }
+        public maxProjectResults?: number,
+        public useEditorFormattingSettings?: boolean) { }
 
     public static Read(): Options {
         // Extra effort is taken below to ensure that legacy versions of options
@@ -37,7 +38,8 @@ export class Options {
 
         const projectLoadTimeout = omnisharpConfig.get<number>('projectLoadTimeout', 60);
         const maxProjectResults = omnisharpConfig.get<number>('maxProjectResults', 250);
+        const useEditorFormattingSettings = omnisharpConfig.get<boolean>('useEditorFormattingSettings', true);
 
-        return new Options(path, useMono, loggingLevel, autoStart, projectLoadTimeout, maxProjectResults);
+        return new Options(path, useMono, loggingLevel, autoStart, projectLoadTimeout, maxProjectResults, useEditorFormattingSettings);
     }
 }