Small suggested cleanups while browsing

Author: andyleejordan

src/PowerShellEditorServices.Hosting/Commands/StartEditorServicesCommand.cs

@@ -15,7 +15,6 @@
 #if DEBUG
 using System.Diagnostics;
 using System.Threading;
-
 using Debugger = System.Diagnostics.Debugger;
 #endif
 
@@ -197,9 +196,11 @@ protected override void BeginProcessing()
 #if DEBUG
             if (WaitForDebugger)
             {
+                // NOTE: Ignore the suggestion to use Environment.ProcessId as it doesn't work for
+                // .NET 4.6.2 (for Windows PowerShell), and this won't be caught in CI.
+                Console.WriteLine($"Waiting for debugger to attach, PID: {Process.GetCurrentProcess().Id}");
                 while (!Debugger.IsAttached)
                 {
-                    Console.WriteLine($"PID: {Process.GetCurrentProcess().Id}");
                     Thread.Sleep(1000);
                 }
             }

src/PowerShellEditorServices/Services/DebugAdapter/Debugging/VariableDetails.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -98,16 +98,11 @@ public VariableDetails(string name, object value)
         /// If this variable instance is expandable, this method returns the
         /// details of its children.  Otherwise it returns an empty array.
         /// </summary>
-        /// <returns></returns>
         public override VariableDetailsBase[] GetChildren(ILogger logger)
         {
             if (IsExpandable)
             {
-                if (cachedChildren == null)
-                {
-                    cachedChildren = GetChildren(ValueObject, logger);
-                }
-
+                cachedChildren ??= GetChildren(ValueObject, logger);
                 return cachedChildren;
             }
 
@@ -131,9 +126,7 @@ private static bool GetIsExpandable(object valueObject)
                 valueObject = psobject.BaseObject;
             }
 
-            Type valueType =
-                valueObject?.GetType();
-
+            Type valueType = valueObject?.GetType();
             TypeInfo valueTypeInfo = valueType.GetTypeInfo();
 
             return
@@ -379,7 +372,7 @@ protected static void AddDotNetProperties(object obj, List<VariableDetails> chil
 
         #endregion
 
-        private struct UnableToRetrievePropertyMessage
+        private readonly struct UnableToRetrievePropertyMessage
         {
             public UnableToRetrievePropertyMessage(string message) => Message = message;
 

src/PowerShellEditorServices/Services/DebugAdapter/Debugging/VariableDetailsBase.cs

@@ -51,7 +51,6 @@ internal abstract class VariableDetailsBase
         /// If this variable instance is expandable, this method returns the
         /// details of its children.  Otherwise it returns an empty array.
         /// </summary>
-        /// <returns></returns>
         public abstract VariableDetailsBase[] GetChildren(ILogger logger);
     }
 }

src/PowerShellEditorServices/Services/DebugAdapter/Handlers/DebugEvaluateHandler.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation.
+// Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
 using System;
@@ -64,21 +64,15 @@ await _executionService.ExecutePSCommandAsync(
                     result = _debugService.GetVariableFromExpression(request.Expression);
 
                     // If the expression is not a naked variable reference, then evaluate the expression.
-                    if (result == null)
-                    {
-                        result =
-                            await _debugService.EvaluateExpressionAsync(
-                                request.Expression,
-                                isFromRepl).ConfigureAwait(false);
-                    }
+                    result ??= await _debugService.EvaluateExpressionAsync(
+                        request.Expression,
+                        isFromRepl).ConfigureAwait(false);
                 }
 
                 if (result != null)
                 {
                     valueString = result.ValueString;
-                    variableId =
-                        result.IsExpandable ?
-                            result.Id : 0;
+                    variableId = result.IsExpandable ? result.Id : 0;
                 }
             }