Handle multiple compile commands on client side (needs native server side support)

Extension/c_cpp_properties.schema.json

@@ -70,9 +70,22 @@
                         ]
                     },
                     "compileCommands": {
-                        "markdownDescription": "Full path to `compile_commands.json` file for the workspace.",
-                        "descriptionHint": "Markdown text between `` should not be translated or localized (they represent literal text) and the capitalization, spacing, and punctuation (including the ``) should not be altered.",
-                        "type": "string"
+                        "oneOf": [
+                            {
+                                "type": "string",
+                                "default": ""
+                            },
+                            {
+                                "type": "array",
+                                "items": {
+                                    "type": "string"
+                                },
+                                "uniqueItems": true,
+                                "default": []
+                            }
+                        ],
+                        "markdownDescription": "Full path or a list of full paths to `compile_commands.json` files for the workspace.",
+                        "descriptionHint": "Markdown text between `` should not be translated or localized (they represent literal text) and the capitalization, spacing, and punctuation (including the ``) should not be altered."
                     },
                     "includePath": {
                         "markdownDescription": "A list of paths for the IntelliSense engine to use while searching for included headers. Searching on these paths is not recursive. Specify `**` to indicate recursive search. For example, `${workspaceFolder}/**` will search through all subdirectories while `${workspaceFolder}` will not. Usually, this should not include system includes; instead, set `C_Cpp.default.compilerPath`.",

Extension/package.json

@@ -693,7 +693,20 @@
                         "scope": "machine-overridable"
                     },
                     "C_Cpp.default.compileCommands": {
-                        "type": "string",
+                        "oneOf": [
+                            {
+                                "type": "string",
+                                "default": ""
+                            },
+                            {
+                                "type": "array",
+                                "items": {
+                                    "type": "string"
+                                },
+                                "uniqueItems": true,
+                                "default": []
+                            }
+                        ],
                         "markdownDescription": "%c_cpp.configuration.default.compileCommands.markdownDescription%",
                         "scope": "machine-overridable"
                     },

Extension/src/LanguageServer/settings.ts

@@ -399,9 +399,26 @@ export class CppSettings extends Settings {
     public get defaultDotconfig(): string | undefined { return changeBlankStringToUndefined(this.getAsStringOrUndefined("default.dotConfig")); }
     public get defaultMacFrameworkPath(): string[] | undefined { return this.getArrayOfStringsWithUndefinedDefault("default.macFrameworkPath"); }
     public get defaultWindowsSdkVersion(): string | undefined { return changeBlankStringToUndefined(this.getAsStringOrUndefined("default.windowsSdkVersion")); }
-    public get defaultCompileCommands(): string | undefined { return changeBlankStringToUndefined(this.getAsStringOrUndefined("default.compileCommands")); }
     public get defaultForcedInclude(): string[] | undefined { return this.getArrayOfStringsWithUndefinedDefault("default.forcedInclude"); }
     public get defaultIntelliSenseMode(): string | undefined { return this.getAsStringOrUndefined("default.intelliSenseMode"); }
+    public get defaultCompileCommands(): string[] | undefined {
+        // Try to get the value as a string.
+        const value: string | undefined = this.getAsStringOrUndefined("default.compileCommands");
+        if (value !== undefined) {
+            if (changeBlankStringToUndefined(value) === undefined) {
+                return undefined;
+            }
+            return [value];
+        }
+
+        // value is not a string, try to get it as an array of strings instead.
+        let valueArray: string[] | undefined = this.getAsArrayOfStringsOrUndefined("default.compileCommands");
+        valueArray = valueArray?.filter((value: string) => value.length > 0);
+        if (valueArray?.length === 0) {
+            return undefined;
+        }
+        return valueArray;
+    }
     public get defaultCompilerPath(): string | null { return this.getAsString("default.compilerPath", true); }
 
     public set defaultCompilerPath(value: string) {
@@ -638,7 +655,7 @@ export class CppSettings extends Settings {
         }
 
         if (isArrayOfString(value)) {
-            if (setting.items.enum && !allowUndefinedEnums) {
+            if (setting.items?.enum !== undefined && !allowUndefinedEnums) {
                 if (!value.every(x => this.isValidEnum(setting.items.enum, x))) {
                     return setting.default;
                 }

Extension/src/LanguageServer/settingsPanel.ts

@@ -337,7 +337,7 @@ export class SettingsPanel {
                 this.configValues.macFrameworkPath = splitEntries(message.value);
                 break;
             case elementId.compileCommands:
-                this.configValues.compileCommands = message.value || undefined;
+                this.configValues.compileCommands = splitEntries(message.value);
                 break;
             case elementId.dotConfig:
                 this.configValues.dotConfig = message.value || undefined;

Extension/ui/settings.html

@@ -672,11 +672,12 @@
         <div class="section">
             <div class="section-title" data-loc-id="compile.commands">Compile commands</div>
             <div class="section-text">
-                <span data-loc-id="compile.commands.description">The full path to the <code>compile_commands.json</code> file for the workspace. The include paths and defines discovered in this file will be used instead of the values set for <code>includePath</code> and <code>defines</code> settings. If the compile commands database does not contain an entry for the translation unit that corresponds to the file you opened in the editor, then a warning message will appear and the extension will use the <code>includePath</code> and <code>defines</code> settings instead.</span>
+                <span data-loc-id="compile.commands.description">A list of paths to <code>compile_commands.json</code> files for the workspace. The include paths and defines discovered in these files will be used instead of the values set for <code>includePath</code> and <code>defines</code> settings. If the compile commands database does not contain an entry for the translation unit that corresponds to the file you opened in the editor, then a warning message will appear and the extension will use the <code>includePath</code> and <code>defines</code> settings instead.</span>
             </div>
             <div>
-                <input name="inputValue" id="compileCommands" style="width: 798px"></input>
-                <div id="compileCommandsInvalid" class="error" style="width: 800px"></div>
+                <div class="section-note" data-loc-id="one.compile.commands.path.per.line">One compile commands path per line.</div>
+                <textarea name="inputValue" id="compileCommands" rows="4" cols="93" style="width: 800px"></textarea>
+                <div id="compileCommandsInvalid" class="error" style="margin-top: -4px; width: 794px"></div>
             </div>
         </div>
 

Extension/ui/settings.ts

@@ -295,7 +295,7 @@ class SettingsApp {
             // Advanced settings
             (<HTMLInputElement>document.getElementById(elementId.windowsSdkVersion)).value = config.windowsSdkVersion ? config.windowsSdkVersion : "";
             (<HTMLInputElement>document.getElementById(elementId.macFrameworkPath)).value = joinEntries(config.macFrameworkPath);
-            (<HTMLInputElement>document.getElementById(elementId.compileCommands)).value = config.compileCommands ? config.compileCommands : "";
+            (<HTMLInputElement>document.getElementById(elementId.compileCommands)).value = joinEntries(config.compileCommands);
             (<HTMLInputElement>document.getElementById(elementId.mergeConfigurations)).checked = config.mergeConfigurations;
             (<HTMLInputElement>document.getElementById(elementId.configurationProvider)).value = config.configurationProvider ? config.configurationProvider : "";
             (<HTMLInputElement>document.getElementById(elementId.forcedInclude)).value = joinEntries(config.forcedInclude);