Fix some comments.

Author: sean-mcmanus

Extension/src/Debugger/configurationProvider.ts

@@ -358,7 +358,7 @@ export class DebugConfigurationProvider implements vscode.DebugConfigurationProv
 
                 // LLDB-DAP supports attaching to process by pid or by name.
                 // If they do specify 'program', then we can check to see if there
-                // is a process that matches, or they are using 'waitFor' (on Linux/OSX).
+                // is a process that matches, or they are using 'waitFor' (on Linux/macOS).
                 if (config.program) {
                     // If they specified 'program', get the tasks that match that.
                     const items = (await attachItemsProvider.getAttachItems(token)).filter(each => processMatches(config.program, each.label, each.fullPath));
@@ -991,12 +991,12 @@ export class DebugConfigurationProvider implements vscode.DebugConfigurationProv
         // Get debug configurations for all debugger types.
         let configs: CppDebugConfiguration[] = [];
         if (isWindows) {
-            // When on windows - prefer cppvsdbg.
+            // When on Windows - prefer cppvsdbg.
             configs.push(...await this.provideDebugConfigurationsForType(DebuggerType.cppvsdbg, folder));
         }
 
         if (!isWindows && await findLldbDap()) {
-            // Only include lldb-dap tasks if it's not windows and lldb-dap is actually available.
+            // Only include lldb-dap tasks if it's not Windows and lldb-dap is actually available.
             configs.push(...await this.provideDebugConfigurationsForType(DebuggerType.cpplldb, folder));
         }
 
@@ -1205,7 +1205,7 @@ export class ConfigurationAssetProvider {
             case 'linux':
                 return new LinuxConfigurationProvider();
             default:
-                return new ConfigurationAssetProvider(); // at least this will return empty results if the platform is not recognized.
+                return new ConfigurationAssetProvider(); // At least this will return empty results if the platform is not recognized.
         }
     }
 }

Extension/src/Debugger/debugAdapterDescriptorFactory.ts

@@ -82,11 +82,11 @@ export class CpplldbDebugAdapterDescriptorFactory extends AbstractDebugAdapterDe
             return null;
         }
 
-        // Prepare the command to run the lldb dap executable.
+        // Prepare the command to run the lldb-dap executable.
         const debuggerArgs = session.configuration.debuggerArgs || [];
 
         // Future: add support for pipeTransport (so that the lldb-dap executable can be run on a remote machine or wsl).
-        // Future: add support for --server mode (so that the lldb dap executable can be run in server mode).
+        // Future: add support for --server mode (so that the lldb-dap executable can be run in server mode).
 
         // Prepare the command to run the lldb-dap executable.
         return new vscode.DebugAdapterExecutable(adapter, [...debuggerArgs]);

Extension/src/Debugger/lldb-dap.ts

@@ -12,11 +12,11 @@ import { CppDebugConfiguration, DebuggerType } from './configurations';
 import { findLldbDapImpl } from './lldb-dap-worker';
 export { isValidLldbDap } from './lldb-dap-worker';
 
-/** Translates the cpplldb configuration to the lldb dap configuration.
+/** Translates the cpplldb configuration to the lldb-dap configuration.
  * Note: this modifies the existing configuration object in place.
  *
  * @param config The cpplldb configuration to translate.
- * @returns The translated lldb dap configuration.
+ * @returns The translated lldb-dap configuration.
  */
 export function translateToLldbDap(config: CppDebugConfiguration) {
     // Adapt the cpplldb config to the lldb-dap config.
@@ -87,9 +87,9 @@ export async function findLldbDap() {
 // This code must only run in the main thread.
 if (isMainThread && !remote) {
     try {
-        // find the entry point for the worker thread.
+        // Find the entry point for the worker thread.
         const file = resolve(__dirname.substring(0, __dirname.lastIndexOf('dist')), "dist", "src", "Debugger", 'lldb-dap-worker.js');
-        // create the worker and connection.
+        // Create the worker and connection.
         remote = startRemoting(startWorker(file), {
             // These are the functions that the main thread exposes to the worker thread.
             log,
@@ -100,7 +100,7 @@ if (isMainThread && !remote) {
         });
 
         if (!isWindows) {
-            // If we are not on Windows, we'll start it searching for the lldb dap executable as early as possible.
+            // If we are not on Windows, we'll start it searching for the lldb-dap executable as early as possible.
             // Mainly, because the LLDB-DAP debugger isn't a common standalone debugger for Windows.
             void findLldbDap();
         }

Extension/src/common-remote-safe.ts

@@ -119,9 +119,9 @@ export function appendLine(message: string | Promise<string>): void {
     }
 }
 
-/** When on windows, ensures that the given executable name ends in an '.exe'.
+/** When on Windows, ensures that the given executable name ends in an '.exe'.
  * @param executableName The name of the executable to check.
- * @returns The executable name with .exe appended if on windows and the name does not already end in an '.exe'.
+ * @returns The executable name with .exe appended if on Windows and the name does not already end in an '.exe'.
  */
 export function executableName(executableName: string) {
     return isWindows && !/\.exe$/i.test(executableName) ? `${executableName}.exe` : executableName;