Added/reviewed some notes

Author: elektronikworkshop

src/deviceContext.ts

@@ -135,12 +135,12 @@ export class DeviceContext implements IDeviceContext, vscode.Disposable {
      *
      * TODO EW, 2020-02-18:
      * A problem I discovered: here you try to find the config file location
-     * and when you're writing below, you use a hardcoded location. When
+     * and when you're writing below, you use a hard-coded location. When
      * resorting to "find", you have to store the file's location at least and
      * reuse it when saving.
      * But I think the intention is: load a config file from anywhere and save
      * it under .vscode/arduino.json. But then the initial load has to use find
-     * and afterwards it must not use it anymore.
+     * and afterwards it must not use find anymore.
      */
     public loadContext(): Thenable<object> {
         return vscode.workspace.findFiles(ARDUINO_CONFIG_FILE, null, 1)
@@ -295,7 +295,7 @@ export class DeviceContext implements IDeviceContext, vscode.Disposable {
                         validateInput: (value) => {
                             /* TODO (EW, 2020-02-18):
                              * is 'c' actually allowed? Also found on within other files.
-                             * And the regular expression doesn't need the intenal groups.
+                             * And the regular expression doesn't need the internal groups.
                              * The outer group can be an anonymous group.
                              * And \w doesn't match dashes - so any sketch containing dashes
                              * will not be found.

src/extension.ts

@@ -189,7 +189,11 @@ export async function activate(context: vscode.ExtensionContext) {
     });
 
     registerArduinoCommand("arduino.selectProgrammer", async () => {
-        // TODO EW: this guard does not prevent building when setting the programmer
+        // Note: this guard does not prevent building while setting the
+        // programmer. But when looking at the code of selectProgrammer
+        // it seems not to be possible to trigger building while setting
+        // the programmer. If the timed IntelliSense analysis is triggered
+        // this is not a problem, since it doesn't use the programmer.
         if (!arduinoContextModule.default.arduinoApp.building) {
             try {
                 await arduinoContextModule.default.arduinoApp.programmerManager.selectProgrammer();