Fix comparison of value type object to null (#13285)

src/Microsoft.PowerShell.Commands.Diagnostics/PdhHelper.cs

@@ -462,7 +462,7 @@ namespace Microsoft.Powershell.Commands.GetCounter.PdhNative
             defaultScale = 0;
             timeBase = 0;
 
-            Debug.Assert(hCounter != null);
+            Debug.Assert(hCounter != IntPtr.Zero);
 
             IntPtr pBufferSize = new IntPtr(0);
             res = PdhGetCounterInfo(hCounter, false, ref pBufferSize, IntPtr.Zero);
@@ -1190,7 +1190,7 @@ namespace Microsoft.Powershell.Commands.GetCounter.PdhNative
                 UInt64 timeBase = 0;
 
                 IntPtr hCounter = _consumerPathToHandleAndInstanceMap[path].hCounter;
-                Debug.Assert(hCounter != null);
+                Debug.Assert(hCounter != IntPtr.Zero);
 
                 res = GetCounterInfoPlus(hCounter, out counterType, out defaultScale, out timeBase);
                 if (res != 0)

src/Microsoft.PowerShell.Commands.Utility/commands/utility/ImplicitRemotingCommands.cs

@@ -1915,7 +1915,7 @@ namespace Microsoft.PowerShell.Commands
             InvocationInfo invocationInfo)
         {
             Dbg.Assert(remoteRunspaceInfo != null, "Caller should validate remoteRunspaceInfo != null");
-            Dbg.Assert(moduleGuid != null, "Caller should validate moduleGuid != null");
+            Dbg.Assert(moduleGuid != Guid.Empty, "Caller should validate moduleGuid is not empty");
             Dbg.Assert(invocationInfo != null, "Caller should validate invocationInfo != null");
 
             _remoteRunspaceInfo = remoteRunspaceInfo;

src/Microsoft.PowerShell.ConsoleHost/host/msh/ConsoleHost.cs

@@ -1206,7 +1206,6 @@ namespace Microsoft.PowerShell
             if (!_isDisposed)
             {
 #if !UNIX
-                Dbg.Assert(breakHandlerGcHandle != null, "break handler should be set");
                 ConsoleControl.RemoveBreakHandler();
                 if (breakHandlerGcHandle.IsAllocated)
                 {

src/System.Management.Automation/FormatAndOutput/common/TableWriter.cs

@@ -204,7 +204,7 @@ namespace Microsoft.PowerShell.Commands.Internal.Format
             int cols = _si.columnInfo.Length;
             Span<int> currentAlignment = cols <= OutCommandInner.StackAllocThreshold ? stackalloc int[cols] : new int[cols];
 
-            if (alignment == null)
+            if (alignment.IsEmpty)
             {
                 for (int i = 0; i < currentAlignment.Length; i++)
                 {

src/System.Management.Automation/engine/remoting/client/RemoteRunspacePoolInternal.cs

@@ -111,9 +111,9 @@ namespace System.Management.Automation.Runspaces.Internal
             ConnectCommandInfo[] connectCommands, RunspaceConnectionInfo connectionInfo, PSHost host, TypeTable typeTable)
             : base(1, 1)
         {
-            if (instanceId == null)
+            if (instanceId == Guid.Empty)
             {
-                throw PSTraceSource.NewArgumentNullException("RunspacePool Guid");
+                throw PSTraceSource.NewArgumentException(nameof(instanceId));
             }
 
             if (connectCommands == null)

src/System.Management.Automation/engine/remoting/fanin/WSManPluginFacade.cs

@@ -318,7 +318,7 @@ namespace System.Management.Automation.Remoting
         private void CleanUpDelegates()
         {
             // Free GCHandles so that the memory they point to may be unpinned (garbage collected)
-            if (_pluginShellGCHandle != null)
+            if (_pluginShellGCHandle.IsAllocated)
             {
                 _pluginShellGCHandle.Free();
                 _pluginReleaseShellContextGCHandle.Free();

src/System.Management.Automation/engine/remoting/fanin/WSManTransportManager.cs

@@ -2280,7 +2280,7 @@ namespace System.Management.Automation.Remoting.Client
             }
 
             // process returned Xml
-            Dbg.Assert(data != null, "WSManConnectShell callback returned null data");
+            Dbg.Assert(data != IntPtr.Zero, "WSManConnectShell callback returned null data");
             WSManNativeApi.WSManConnectDataResult connectData = WSManNativeApi.WSManConnectDataResult.UnMarshal(data);
             if (connectData.data != null)
             {

src/System.Management.Automation/help/CabinetNativeApi.cs

@@ -152,7 +152,7 @@ namespace System.Management.Automation.Internal
         private void CleanUpDelegates()
         {
             // Free GCHandles so that the memory they point to may be unpinned (garbage collected)
-            if (_fdiAllocHandle != null)
+            if (_fdiAllocHandle.IsAllocated)
             {
                 _fdiAllocHandle.Free();
                 _fdiFreeHandle.Free();

src/System.Management.Automation/help/UpdatableHelpModuleInfo.cs

@@ -28,7 +28,7 @@ namespace System.Management.Automation.Help
         internal UpdatableHelpModuleInfo(string name, Guid guid, string path, string uri)
         {
             Debug.Assert(!string.IsNullOrEmpty(name));
-            Debug.Assert(guid != null);
+            Debug.Assert(guid != Guid.Empty);
             Debug.Assert(!string.IsNullOrEmpty(path));
             Debug.Assert(!string.IsNullOrEmpty(uri));
 

src/System.Management.Automation/help/UpdatableHelpUri.cs

@@ -21,7 +21,7 @@ namespace System.Management.Automation.Help
         internal UpdatableHelpUri(string moduleName, Guid moduleGuid, CultureInfo culture, string resolvedUri)
         {
             Debug.Assert(!string.IsNullOrEmpty(moduleName));
-            Debug.Assert(moduleGuid != null);
+            Debug.Assert(moduleGuid != Guid.Empty);
             Debug.Assert(!string.IsNullOrEmpty(resolvedUri));
 
             ModuleName = moduleName;

src/System.Management.Automation/security/Authenticode.cs

@@ -192,7 +192,7 @@ namespace System.Management.Automation
                     IntPtr.Zero);
 #pragma warning restore 56523
 
-                if (si.pSignExtInfo != null)
+                if (si.pSignExtInfo != IntPtr.Zero)
                 {
                     Marshal.DestroyStructure<NativeMethods.CRYPTUI_WIZ_DIGITAL_SIGN_EXTENDED_INFO>(si.pSignExtInfo);
                     Marshal.FreeCoTaskMem(si.pSignExtInfo);