Skip to content

Commit 18a108a

Browse files
committed
Fix comment/string casing issues.
1 parent 998b257 commit 18a108a

File tree

4 files changed

+11
-11
lines changed

4 files changed

+11
-11
lines changed

Extension/src/Debugger/configurationProvider.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -371,7 +371,7 @@ export class DebugConfigurationProvider implements vscode.DebugConfigurationProv
371371
// On Windows, waitFor on lldb-dap is not supported - so we'll let the user know.
372372
void logger.getOutputChannelLogger().showWarningMessage(localize("waitFor.not.supported", "The {0} debugger does not support '{1}' on Windows.", "LLDB-DAP", "waitFor"));
373373

374-
// Drop the waitFor and program (which will just have the picker show all processes)
374+
// Drop the waitFor and program (which will just have the picker show all processes).
375375
delete config.program;
376376
delete config.waitFor;
377377
}

Extension/src/Debugger/lldb-dap-worker.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ let lldbDapPath: string | undefined;
9393
let xcRunPath: string | undefined;
9494

9595
/**
96-
* On MacOS, calls xcrun to find a given xtools binary (usually they don't expose the binary to the PATH).
96+
* On macOS, calls xcrun to find a given xtools binary (usually they don't expose the binary to the PATH).
9797
*
9898
* @param filename The name of the binary to find (e.g., lldb-vscode or lldb-dap).
9999
* @returns The path to the binary or undefined if it was not found.
@@ -139,7 +139,7 @@ const candidates = [
139139
*
140140
* We absolutely prefer an lldb-dap binary that is in the path.
141141
* Failing that, we'll:
142-
* - Try xcrun on MacOS to see if xtools can give it to us
142+
* - Try xcrun on macOS to see if xtools can give it to us
143143
* - Try searching the secure well-known locations for a binary
144144
*
145145
* And if we can't find the actual lldb-dap binary, we'll try the same thing
@@ -171,7 +171,7 @@ async function searchForLldbDap() {
171171
if (typeof candidate === 'string') {
172172
lldbDapPath = await xcRun(candidate);
173173
if (lldbDapPath) {
174-
appendLineAtLevel(6, `Discovered lldb-dap binary for MacOS at '${lldbDapPath}' ${Date.now() - start} msec`);
174+
appendLineAtLevel(6, `Discovered lldb-dap binary for macOS at '${lldbDapPath}' ${Date.now() - start} msec`);
175175
return lldbDapPath;
176176
}
177177
}
@@ -180,7 +180,7 @@ async function searchForLldbDap() {
180180
// we'll do a check of well known secured locations where it might be installed.
181181
lldbDapPath = await searchFolders(['/Applications', '/opt/homebrew'], candidate, isValidLldbDap, 8);
182182
if (lldbDapPath) {
183-
appendLineAtLevel(6, `Discovered lldb-dap binary for MacOS at '${lldbDapPath}' ${Date.now() - start} msec`);
183+
appendLineAtLevel(6, `Discovered lldb-dap binary for macOS at '${lldbDapPath}' ${Date.now() - start} msec`);
184184
return lldbDapPath;
185185
}
186186
}

Extension/src/Debugger/nativeAttach.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ export class PsAttachItemsProvider extends NativeAttachItemsProvider {
123123
}
124124

125125
export class PsProcessParser {
126-
// Use a large fixed width - the default on MacOS is quite small.
126+
// Use a large fixed width - the default on macOS is quite small.
127127
static fixedWidth = ''.padEnd(512, 'a');
128128

129129
// Note that comm on Linux systems is truncated to 16 characters:
@@ -159,11 +159,11 @@ export class PsProcessParser {
159159

160160
private static parseLineFromPs(line: string): Process | undefined {
161161
const psEntry = isMacOS ?
162-
// ON MacOS, we're using fixed-width columns, so we have to use a fixed-width regex.
162+
// On macOS, we're using fixed-width columns, so we have to use a fixed-width regex.
163163
// <start>whitespace(NUMBERS)whitespace(FIXED-WIDTH)whitespace(EVERYTHING-ELSE)<end>
164164
new RegExp(`^\\s*([0-9]+)\\s+(.{${PsProcessParser.fixedWidth.length - 1}})\\s+(.*)$`) :
165165

166-
// on Linux, column widths cannot be guaranteed - but we do get escaped spaces in the command line.
166+
// On Linux, column widths cannot be guaranteed - but we do get escaped spaces in the command line.
167167
// <start>whitespace(NUMBERS)whitespace(NOTWHITESPACE)whitespace(EVERYTHING-ELSE)<end>
168168
/^\s*(\d+)\s+((?:\\\s|\S)+)\s+([\s\S]*)$/;
169169

Extension/src/common-remote-safe.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -363,7 +363,7 @@ export async function isFolder(path: string) {
363363
try {
364364
return (await stat(path)).isDirectory();
365365
} catch {
366-
// ignore errors, if we can't access the path, it's not a folder.
366+
// Ignore errors, if we can't access the path, it's not a folder.
367367
}
368368
return false;
369369
}
@@ -374,7 +374,7 @@ export async function searchFolders(folders: string[], filename: string | RegExp
374374
const nameMatches = is.string(filename) ? (item: string) => item === filename : (item: string) => filename.test(item);
375375

376376
for (let folder of folders) {
377-
// if the result was reached from anything going on asynchronously, we can stop searching.
377+
// If the result was reached from anything going on asynchronously, we can stop searching.
378378
if (options.result) {
379379
return options.result;
380380
}
@@ -385,7 +385,7 @@ export async function searchFolders(folders: string[], filename: string | RegExp
385385
continue;
386386
}
387387

388-
// if we've been here before, skip this folder.
388+
// If we've been here before, skip this folder.
389389
if (options.visited.has(folder) || !await isFolder(folder)) {
390390
continue;
391391
}

0 commit comments

Comments
 (0)