Changed Serial.updatePortListStatus to make use of optional argument to eliminate updatePortListStatus(null) calls

Author: elektronikworkshop

src/serialmonitor/serialMonitor.ts

@@ -73,7 +73,7 @@ export class SerialMonitor implements vscode.Disposable {
         this._baudRateStatusBar.command = "arduino.changeBaudRate";
         this._baudRateStatusBar.tooltip = "Baud Rate";
         this._baudRateStatusBar.text = defaultBaudRate.toString();
-        this.updatePortListStatus(null);
+        this.updatePortListStatus();
 
         this._endingStatusBar = vscode.window.createStatusBarItem(vscode.StatusBarAlignment.Right, constants.statusBarPriority.ENDING);
         this._ending = SerialMonitor.DEFAULT_ENDING;
@@ -83,7 +83,7 @@ export class SerialMonitor implements vscode.Disposable {
 
         const dc = DeviceContext.getInstance();
         dc.onChangePort(() => {
-            this.updatePortListStatus(null);
+            this.updatePortListStatus();
         });
     }
     public get initialized(): boolean {
@@ -230,8 +230,7 @@ export class SerialMonitor implements vscode.Disposable {
         }
     }
 
-    // TODO EW: use default value for port function parameter and change all updatePortListStatus(null) calls accordingly
-    private updatePortListStatus(port: string) {
+    private updatePortListStatus(port?: string) {
         const dc = DeviceContext.getInstance();
         if (port) {
             dc.port = port;