Use is not syntax where appropriate and remove unnecessary parentheses (PowerShell#13323)

xtqqczze · web-flow · commit 4597b41396f8 · 2020-07-31T16:41:59.000-07:00
diff --git a/src/Microsoft.Management.Infrastructure.CimCmdlets/CimAsyncOperation.cs b/src/Microsoft.Management.Infrastructure.CimCmdlets/CimAsyncOperation.cs
@@ -404,7 +404,7 @@ protected object GetReferenceOrReferenceArrayObject(object value, ref CimType re
                 {
                     return null;
                 }
-                else if (!(cimReferenceArray[0] is PSReference))
+                else if (cimReferenceArray[0] is not PSReference)
                 {
                     return null;
                 }
diff --git a/src/Microsoft.Management.Infrastructure.CimCmdlets/CimSessionProxy.cs b/src/Microsoft.Management.Infrastructure.CimCmdlets/CimSessionProxy.cs
@@ -1055,7 +1055,7 @@ private static void AddShowComputerNameMarker(object o)
             }
 
             PSObject pso = PSObject.AsPSObject(o);
-            if (!(pso.BaseObject is CimInstance))
+            if (pso.BaseObject is not CimInstance)
             {
                 return;
             }
@@ -2086,7 +2086,7 @@ protected override bool PreNewActionEvent(CmdletActionEventArgs args)
         {
             DebugHelper.WriteLogEx();
 
-            if (!(args.Action is CimWriteResultObject))
+            if (args.Action is not CimWriteResultObject)
             {
                 // allow all other actions
                 return true;
@@ -2240,7 +2240,7 @@ protected override bool PreNewActionEvent(CmdletActionEventArgs args)
         {
             DebugHelper.WriteLogEx();
 
-            if (!(args.Action is CimWriteResultObject))
+            if (args.Action is not CimWriteResultObject)
             {
                 // allow all other actions
                 return true;
diff --git a/src/Microsoft.PowerShell.Commands.Management/cimSupport/cmdletization/cim/clientSideQuery.cs b/src/Microsoft.PowerShell.Commands.Management/cimSupport/cmdletization/cim/clientSideQuery.cs
@@ -405,7 +405,7 @@ public bool IsMatch(CimInstance o)
 
             private object ConvertActualValueToExpectedType(object actualPropertyValue, object expectedPropertyValue)
             {
-                if ((actualPropertyValue is string) && (!(expectedPropertyValue is string)))
+                if (actualPropertyValue is string && expectedPropertyValue is not string)
                 {
                     actualPropertyValue = LanguagePrimitives.ConvertTo(actualPropertyValue, expectedPropertyValue.GetType(), CultureInfo.InvariantCulture);
                 }
diff --git a/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs b/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs
@@ -667,7 +667,7 @@ internal virtual void PrepareSession()
 
                     // null is not valid value for header.
                     // We silently ignore header if value is null.
-                    if (!(value is null))
+                    if (value is not null)
                     {
                         // add the header value (or overwrite it if already present)
                         WebSession.Headers[key] = value.ToString();
diff --git a/src/Microsoft.PowerShell.Commands.Utility/commands/utility/trace/TraceExpressionCommand.cs b/src/Microsoft.PowerShell.Commands.Utility/commands/utility/trace/TraceExpressionCommand.cs
@@ -525,7 +525,7 @@ private static ErrorRecord ConvertToErrorRecord(object obj)
             if (mshobj != null)
             {
                 object baseObject = mshobj.BaseObject;
-                if (!(baseObject is PSCustomObject))
+                if (baseObject is not PSCustomObject)
                 {
                     obj = baseObject;
                 }
diff --git a/src/Microsoft.PowerShell.ConsoleHost/WindowsTaskbarJumpList/PropertyKey.cs b/src/Microsoft.PowerShell.ConsoleHost/WindowsTaskbarJumpList/PropertyKey.cs
@@ -75,7 +75,7 @@ public override bool Equals(object obj)
             if (obj == null)
                 return false;
 
-            if (!(obj is PropertyKey))
+            if (obj is not PropertyKey)
                 return false;
 
             PropertyKey other = (PropertyKey)obj;
diff --git a/src/Microsoft.PowerShell.ScheduledJob/ScheduledJobDefinition.cs b/src/Microsoft.PowerShell.ScheduledJob/ScheduledJobDefinition.cs
@@ -938,7 +938,7 @@ public void Register()
                     { }
                 }
 
-                if (!(ex is ScheduledJobException))
+                if (ex is not ScheduledJobException)
                 {
                     // Wrap in ScheduledJobException type.
                     string msg = StringUtil.Format(ScheduledJobErrorStrings.ErrorRegisteringDefinitionStore, this.Name);
diff --git a/src/Microsoft.PowerShell.Security/security/AclCommands.cs b/src/Microsoft.PowerShell.Security/security/AclCommands.cs
@@ -783,7 +783,7 @@ protected override void ProcessRecord()
                     {
                         customDescriptor = PSObject.Base(methodInfo.Invoke());
 
-                        if (!(customDescriptor is FileSystemSecurity))
+                        if (customDescriptor is not FileSystemSecurity)
                         {
                             customDescriptor = new CommonSecurityDescriptor(false, false, customDescriptor.ToString());
                         }
diff --git a/src/Microsoft.PowerShell.Security/security/CertificateProvider.cs b/src/Microsoft.PowerShell.Security/security/CertificateProvider.cs
@@ -1195,7 +1195,7 @@ protected override bool ItemExists(string path)
                     // If the inner exception is not of that type
                     // then we need to rethrow
                     //
-                    if (!(e.InnerException is CertificateProviderItemNotFoundException))
+                    if (e.InnerException is not CertificateProviderItemNotFoundException)
                     {
                         throw;
                     }
diff --git a/src/System.Management.Automation/FormatAndOutput/out-console/OutConsole.cs b/src/System.Management.Automation/FormatAndOutput/out-console/OutConsole.cs
@@ -109,11 +109,10 @@ protected override void ProcessRecord()
                 object inputObjectBase = PSObject.Base(InputObject);
 
                 // Ignore errors and formatting records, as those can't be captured
-                if (
-                    (inputObjectBase != null) &&
-                    (!(inputObjectBase is ErrorRecord)) &&
-                    (!inputObjectBase.GetType().FullName.StartsWith(
-                        "Microsoft.PowerShell.Commands.Internal.Format", StringComparison.OrdinalIgnoreCase)))
+                if (inputObjectBase != null &&
+                    inputObjectBase is not ErrorRecord &&
+                    !inputObjectBase.GetType().FullName.StartsWith(
+                        "Microsoft.PowerShell.Commands.Internal.Format", StringComparison.OrdinalIgnoreCase))
                 {
                     _outVarResults.Add(InputObject);
                 }
diff --git a/src/System.Management.Automation/engine/CommandCompletion/CompletionAnalysis.cs b/src/System.Management.Automation/engine/CommandCompletion/CompletionAnalysis.cs
@@ -1805,7 +1805,7 @@ private List<CompletionResult> GetResultForIdentifier(CompletionContext completi
                             }
                         }
                         // Continue trying the filename/commandname completion for scenarios like this: $aa[get-<tab>
-                        else if (!(cursorAst is ErrorExpressionAst && cursorAst.Parent is IndexExpressionAst))
+                        else if (cursorAst is not ErrorExpressionAst || cursorAst.Parent is not IndexExpressionAst)
                         {
                             return result;
                         }
diff --git a/src/System.Management.Automation/engine/CommandCompletion/CompletionCompleters.cs b/src/System.Management.Automation/engine/CommandCompletion/CompletionCompleters.cs
@@ -1895,7 +1895,7 @@ private static IEnumerable<PSTypeName> NativeCommandArgumentCompletion_InferType
                             yield return new PSTypeName(pso.TypeNames[0]);
                         }
 
-                        if (!(pso.BaseObject is PSCustomObject))
+                        if (pso.BaseObject is not PSCustomObject)
                         {
                             yield return new PSTypeName(pso.BaseObject.GetType());
                         }
@@ -7140,7 +7140,7 @@ IEnumerable<CompletionResult> IArgumentCompleter.CompleteArgument(
             CommandAst commandAst,
             IDictionary fakeBoundParameters)
         {
-            if (!(commandAst.Parent is PipelineAst pipelineAst))
+            if (commandAst.Parent is not PipelineAst pipelineAst)
             {
                 return null;
             }
diff --git a/src/System.Management.Automation/engine/CompiledCommandParameter.cs b/src/System.Management.Automation/engine/CompiledCommandParameter.cs
@@ -69,7 +69,7 @@ internal CompiledCommandParameter(RuntimeDefinedParameter runtimeDefinedParamete
                     }
                 }
 
-                if (!(attribute is ArgumentTypeConverterAttribute))
+                if (attribute is not ArgumentTypeConverterAttribute)
                 {
                     ProcessAttribute(runtimeDefinedParameter.Name, attribute, ref validationAttributes, ref argTransformationAttributes, ref aliases);
                 }
diff --git a/src/System.Management.Automation/engine/EventManager.cs b/src/System.Management.Automation/engine/EventManager.cs
@@ -2597,7 +2597,7 @@ internal void Invoke(PSEventSubscriber eventSubscriber, PSEventArgs eventArgs)
             catch (Exception e)
             {
                 // Catch-all OK. This is a third-party call-out.
-                if (!(e is PipelineStoppedException))
+                if (e is not PipelineStoppedException)
                 {
                     LogErrorsAndOutput(results, actionState);
                     SetJobState(JobState.Failed);
diff --git a/src/System.Management.Automation/engine/LanguagePrimitives.cs b/src/System.Management.Automation/engine/LanguagePrimitives.cs
@@ -906,7 +906,7 @@ public static bool TryCompare(object first, object second, bool ignoreCase, IFor
                 formatProvider = CultureInfo.InvariantCulture;
             }
 
-            if (!(formatProvider is CultureInfo culture))
+            if (formatProvider is not CultureInfo culture)
             {
                 throw PSTraceSource.NewArgumentException(nameof(formatProvider));
             }
@@ -936,7 +936,7 @@ public static bool TryCompare(object first, object second, bool ignoreCase, IFor
 
             if (first is string firstString)
             {
-                if (!(second is string secondString))
+                if (second is not string secondString)
                 {
                     if (!TryConvertTo(second, culture, out secondString))
                     {
@@ -4283,7 +4283,7 @@ public override int GetHashCode()
 
             public override bool Equals(object other)
             {
-                if (!(other is ConversionTypePair))
+                if (other is not ConversionTypePair)
                     return false;
 
                 var ctp = (ConversionTypePair)other;
diff --git a/src/System.Management.Automation/engine/Modules/NewModuleManifestCommand.cs b/src/System.Management.Automation/engine/Modules/NewModuleManifestCommand.cs
@@ -628,7 +628,7 @@ private IEnumerable PreProcessModuleSpec(IEnumerable moduleSpecs)
             {
                 foreach (object spec in moduleSpecs)
                 {
-                    if (!(spec is Hashtable))
+                    if (spec is not Hashtable)
                     {
                         yield return spec.ToString();
                     }
diff --git a/src/System.Management.Automation/engine/MshCommandRuntime.cs b/src/System.Management.Automation/engine/MshCommandRuntime.cs
@@ -953,7 +953,7 @@ internal void SetupOutVariable()
 
                 _outVarList = oldValue ?? new ArrayList();
 
-                if (!(_thisCommand is PSScriptCmdlet))
+                if (_thisCommand is not PSScriptCmdlet)
                 {
                     this.OutputPipe.AddVariableList(VariableStreamKind.Output, _outVarList);
                 }
@@ -989,7 +989,7 @@ internal void SetupPipelineVariable()
             // same scope.
             _pipelineVarReference = _state.PSVariable.Get(this.PipelineVariable);
 
-            if (!(_thisCommand is PSScriptCmdlet))
+            if (_thisCommand is not PSScriptCmdlet)
             {
                 this.OutputPipe.SetPipelineVariable(_pipelineVarReference);
             }
@@ -2574,7 +2574,7 @@ internal void SetupVariable(VariableStreamKind streamKind, string variableName,
                 varList = new ArrayList();
             }
 
-            if (!(_thisCommand is PSScriptCmdlet))
+            if (_thisCommand is not PSScriptCmdlet)
             {
                 this.OutputPipe.AddVariableList(streamKind, varList);
             }
diff --git a/src/System.Management.Automation/engine/MshMemberInfo.cs b/src/System.Management.Automation/engine/MshMemberInfo.cs
@@ -183,7 +183,7 @@ internal virtual void ReplicateInstance(object particularInstance)
 
         internal void SetValueNoConversion(object setValue)
         {
-            if (!(this is PSProperty thisAsProperty))
+            if (this is not PSProperty thisAsProperty)
             {
                 this.Value = setValue;
                 return;
@@ -552,7 +552,7 @@ private void LookupMember(string name, HashSet<string> visitedAliases, out PSMem
                     name);
             }
 
-            if (!(member is PSAliasProperty aliasMember))
+            if (member is not PSAliasProperty aliasMember)
             {
                 hasCycle = false;
                 returnedMember = member;
diff --git a/src/System.Management.Automation/engine/TypeTable.cs b/src/System.Management.Automation/engine/TypeTable.cs
@@ -2191,7 +2191,7 @@ public TypeMemberData StringSerializationSourceProperty
                     return;
                 }
 
-                if (!(value is NotePropertyData || value is ScriptPropertyData || value is CodePropertyData))
+                if (value is not NotePropertyData && value is not ScriptPropertyData && value is not CodePropertyData)
                 {
                     throw PSTraceSource.NewArgumentException("value");
                 }
diff --git a/src/System.Management.Automation/engine/debugger/debugger.cs b/src/System.Management.Automation/engine/debugger/debugger.cs
@@ -4945,7 +4945,7 @@ private string FixUpStatementExtent(int startColNum, string stateExtentText)
         private object DrainAndBlockRemoteOutput()
         {
             // We only do this for remote runspaces.
-            if (!(_runspace is RemoteRunspace)) { return null; }
+            if (_runspace is not RemoteRunspace) { return null; }
 
             try
             {
diff --git a/src/System.Management.Automation/engine/hostifaces/ConnectionBase.cs b/src/System.Management.Automation/engine/hostifaces/ConnectionBase.cs
@@ -268,7 +268,7 @@ private void CoreOpen(bool syncCall)
             // sometimes there are many runspaces created - the callee ensures telemetry is only
             // reported once. Note that if the host implements IHostProvidesTelemetryData, we rely
             // on the host calling ReportStartupTelemetry.
-            if (!(this.Host is IHostProvidesTelemetryData))
+            if (this.Host is not IHostProvidesTelemetryData)
             {
                 TelemetryAPI.ReportStartupTelemetry(null);
             }
diff --git a/src/System.Management.Automation/engine/hostifaces/ConnectionFactory.cs b/src/System.Management.Automation/engine/hostifaces/ConnectionFactory.cs
@@ -454,11 +454,11 @@ public static RunspacePool CreateRunspacePool(int minRunspaces,
         public static RunspacePool CreateRunspacePool(int minRunspaces,
             int maxRunspaces, RunspaceConnectionInfo connectionInfo, PSHost host, TypeTable typeTable, PSPrimitiveDictionary applicationArguments)
         {
-            if ((!(connectionInfo is WSManConnectionInfo)) &&
-                (!(connectionInfo is NewProcessConnectionInfo)) &&
-                (!(connectionInfo is NamedPipeConnectionInfo)) &&
-                (!(connectionInfo is VMConnectionInfo)) &&
-                (!(connectionInfo is ContainerConnectionInfo)))
+            if (connectionInfo is not WSManConnectionInfo &&
+                connectionInfo is not NewProcessConnectionInfo &&
+                connectionInfo is not NamedPipeConnectionInfo &&
+                connectionInfo is not VMConnectionInfo &&
+                connectionInfo is not ContainerConnectionInfo)
             {
                 throw new NotSupportedException();
             }
@@ -539,12 +539,12 @@ public static Runspace CreateRunspace(RunspaceConnectionInfo connectionInfo, PSH
         /// <returns></returns>
         public static Runspace CreateRunspace(RunspaceConnectionInfo connectionInfo, PSHost host, TypeTable typeTable, PSPrimitiveDictionary applicationArguments, string name)
         {
-            if ((!(connectionInfo is WSManConnectionInfo)) &&
-                (!(connectionInfo is NewProcessConnectionInfo)) &&
-                (!(connectionInfo is NamedPipeConnectionInfo)) &&
-                (!(connectionInfo is SSHConnectionInfo)) &&
-                (!(connectionInfo is VMConnectionInfo)) &&
-                (!(connectionInfo is ContainerConnectionInfo)))
+            if (connectionInfo is not WSManConnectionInfo &&
+                connectionInfo is not NewProcessConnectionInfo &&
+                connectionInfo is not NamedPipeConnectionInfo &&
+                connectionInfo is not SSHConnectionInfo &&
+                connectionInfo is not VMConnectionInfo &&
+                connectionInfo is not ContainerConnectionInfo)
             {
                 throw new NotSupportedException();
             }
@@ -606,4 +606,3 @@ public static Runspace CreateOutOfProcessRunspace(TypeTable typeTable, PowerShel
         #endregion V3 Extensions
     }
 }
-
diff --git a/src/System.Management.Automation/engine/hostifaces/History.cs b/src/System.Management.Automation/engine/hostifaces/History.cs
@@ -1459,7 +1459,7 @@ void ProcessRecord()
                 {
                     PSObject serializedPipelineState = pipelineState as PSObject;
                     object baseObject = serializedPipelineState.BaseObject;
-                    if (!(baseObject is int))
+                    if (baseObject is not int)
                     {
                         break;
                     }
diff --git a/src/System.Management.Automation/engine/hostifaces/PSDataCollection.cs b/src/System.Management.Automation/engine/hostifaces/PSDataCollection.cs
@@ -1634,7 +1634,7 @@ private static void VerifyValueType(object value)
                     throw PSTraceSource.NewArgumentNullException(nameof(value), PSDataBufferStrings.ValueNullReference);
                 }
             }
-            else if (!(value is T))
+            else if (value is not T)
             {
                 throw PSTraceSource.NewArgumentException(nameof(value), PSDataBufferStrings.CannotConvertToGenericType,
                                                          value.GetType().FullName,
diff --git a/src/System.Management.Automation/engine/hostifaces/RunspacePool.cs b/src/System.Management.Automation/engine/hostifaces/RunspacePool.cs
@@ -638,7 +638,7 @@ internal RunspacePool(
             TypeTable typeTable)
         {
             // Disconnect-Connect semantics are currently only supported in WSMan transport.
-            if (!(connectionInfo is WSManConnectionInfo))
+            if (connectionInfo is not WSManConnectionInfo)
             {
                 throw new NotSupportedException();
             }
diff --git a/src/System.Management.Automation/engine/interpreter/CallInstruction.cs b/src/System.Management.Automation/engine/interpreter/CallInstruction.cs
@@ -104,7 +104,7 @@ public static CallInstruction Create(MethodInfo info, ParameterInfo[] parameters
             }
             catch (TargetInvocationException tie)
             {
-                if (!(tie.InnerException is NotSupportedException))
+                if (tie.InnerException is not NotSupportedException)
                 {
                     throw;
                 }
diff --git a/src/System.Management.Automation/engine/parser/Compiler.cs b/src/System.Management.Automation/engine/parser/Compiler.cs
@@ -3683,7 +3683,7 @@ public object VisitPipeline(PipelineAst pipelineAst)
             var temps = new List<ParameterExpression>();
             var exprs = new List<Expression>();
 
-            if (!(pipelineAst.Parent is AssignmentStatementAst || pipelineAst.Parent is ParenExpressionAst))
+            if (pipelineAst.Parent is not AssignmentStatementAst && pipelineAst.Parent is not ParenExpressionAst)
             {
                 // If the parent is an assignment, we've already added a sequence point, don't add another.
                 exprs.Add(UpdatePosition(pipelineAst));
diff --git a/src/System.Management.Automation/engine/parser/Parser.cs b/src/System.Management.Automation/engine/parser/Parser.cs
@@ -2423,7 +2423,7 @@ private StatementAst IfStatementRule(Token ifToken)
 
                     UngetToken(rParen);
                     // Don't bother reporting this error if we already reported an empty condition error.
-                    if (!(condition is ErrorStatementAst))
+                    if (condition is not ErrorStatementAst)
                     {
                         ReportIncompleteInput(rParen.Extent,
                             nameof(ParserStrings.MissingEndParenthesisAfterStatement),
@@ -3611,7 +3611,7 @@ private StatementAst WhileStatementRule(LabelToken labelToken, Token whileToken)
                 // so stop parsing the statement and try parsing something else if possible.
 
                 UngetToken(rParen);
-                if (!(condition is ErrorStatementAst))
+                if (condition is not ErrorStatementAst)
                 {
                     ReportIncompleteInput(After(condition),
                         nameof(ParserStrings.MissingEndParenthesisAfterStatement),
@@ -4966,7 +4966,7 @@ private StatementAst UsingStatementRule(Token usingToken)
                     {
                         htAst = (HashtableAst)aliasAst;
                     }
-                    else if (!(aliasAst is StringConstantExpressionAst))
+                    else if (aliasAst is not StringConstantExpressionAst)
                     {
                         return new ErrorStatementAst(ExtentOf(usingToken, aliasAst), new Ast[] { itemAst, aliasAst });
                     }
@@ -6703,7 +6703,7 @@ private ExpressionAst ExpressionRule(bool endNumberOnTernaryOpChars = false)
                     UngetToken(token);
 
                     // Don't bother reporting this error if we already reported an empty 'IfTrue' operand error.
-                    if (!(ifTrue is ErrorExpressionAst))
+                    if (ifTrue is not ErrorExpressionAst)
                     {
                         componentAsts.Add(ifTrue);
                         ReportIncompleteInput(
@@ -7852,7 +7852,7 @@ private ExpressionAst ElementAccessRule(ExpressionAst primaryExpression, Token l
 
                 UngetToken(rBracket);
                 // Skip reporting the error if we've already reported a missing index.
-                if (!(indexExpr is ErrorExpressionAst))
+                if (indexExpr is not ErrorExpressionAst)
                 {
                     ReportIncompleteInput(After(indexExpr),
                         nameof(ParserStrings.MissingEndSquareBracket),
diff --git a/src/System.Management.Automation/engine/parser/SemanticChecks.cs b/src/System.Management.Automation/engine/parser/SemanticChecks.cs
diff --git a/src/System.Management.Automation/engine/parser/SymbolResolver.cs b/src/System.Management.Automation/engine/parser/SymbolResolver.cs
diff --git a/src/System.Management.Automation/engine/parser/TypeInferenceVisitor.cs b/src/System.Management.Automation/engine/parser/TypeInferenceVisitor.cs
diff --git a/src/System.Management.Automation/engine/parser/tokenizer.cs b/src/System.Management.Automation/engine/parser/tokenizer.cs
diff --git a/src/System.Management.Automation/engine/pipeline.cs b/src/System.Management.Automation/engine/pipeline.cs
diff --git a/src/System.Management.Automation/engine/remoting/client/Job.cs b/src/System.Management.Automation/engine/remoting/client/Job.cs
diff --git a/src/System.Management.Automation/engine/remoting/fanin/InitialSessionStateProvider.cs b/src/System.Management.Automation/engine/remoting/fanin/InitialSessionStateProvider.cs
diff --git a/src/System.Management.Automation/engine/runtime/Binding/Binders.cs b/src/System.Management.Automation/engine/runtime/Binding/Binders.cs
diff --git a/src/System.Management.Automation/engine/runtime/CompiledScriptBlock.cs b/src/System.Management.Automation/engine/runtime/CompiledScriptBlock.cs
diff --git a/src/System.Management.Automation/engine/runtime/Operations/MiscOps.cs b/src/System.Management.Automation/engine/runtime/Operations/MiscOps.cs
diff --git a/src/System.Management.Automation/engine/serialization.cs b/src/System.Management.Automation/engine/serialization.cs
diff --git a/src/System.Management.Automation/help/CommandHelpProvider.cs b/src/System.Management.Automation/help/CommandHelpProvider.cs
diff --git a/src/System.Management.Automation/help/HelpSystem.cs b/src/System.Management.Automation/help/HelpSystem.cs
diff --git a/src/System.Management.Automation/namespaces/FileSystemProvider.cs b/src/System.Management.Automation/namespaces/FileSystemProvider.cs

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,7 @@ protected object GetReferenceOrReferenceArrayObject(object value, ref CimType re`
`404`	`404`	`{`
`405`	`405`	`return null;`
`406`	`406`	`}`
`407`		`- else if (!(cimReferenceArray[0] is PSReference))`
	`407`	`+ else if (cimReferenceArray[0] is not PSReference)`
`408`	`408`	`{`
`409`	`409`	`return null;`
`410`	`410`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1055,7 +1055,7 @@ private static void AddShowComputerNameMarker(object o)`
`1055`	`1055`	`}`
`1056`	`1056`
`1057`	`1057`	`PSObject pso = PSObject.AsPSObject(o);`
`1058`		`- if (!(pso.BaseObject is CimInstance))`
	`1058`	`+ if (pso.BaseObject is not CimInstance)`
`1059`	`1059`	`{`
`1060`	`1060`	`return;`
`1061`	`1061`	`}`
`@@ -2086,7 +2086,7 @@ protected override bool PreNewActionEvent(CmdletActionEventArgs args)`
`2086`	`2086`	`{`
`2087`	`2087`	`DebugHelper.WriteLogEx();`
`2088`	`2088`
`2089`		`- if (!(args.Action is CimWriteResultObject))`
	`2089`	`+ if (args.Action is not CimWriteResultObject)`
`2090`	`2090`	`{`
`2091`	`2091`	`// allow all other actions`
`2092`	`2092`	`return true;`
`@@ -2240,7 +2240,7 @@ protected override bool PreNewActionEvent(CmdletActionEventArgs args)`
`2240`	`2240`	`{`
`2241`	`2241`	`DebugHelper.WriteLogEx();`
`2242`	`2242`
`2243`		`- if (!(args.Action is CimWriteResultObject))`
	`2243`	`+ if (args.Action is not CimWriteResultObject)`
`2244`	`2244`	`{`
`2245`	`2245`	`// allow all other actions`
`2246`	`2246`	`return true;`
Original file line number	Diff line number	Diff line change
`@@ -405,7 +405,7 @@ public bool IsMatch(CimInstance o)`
`405`	`405`
`406`	`406`	`private object ConvertActualValueToExpectedType(object actualPropertyValue, object expectedPropertyValue)`
`407`	`407`	`{`
`408`		`- if ((actualPropertyValue is string) && (!(expectedPropertyValue is string)))`
	`408`	`+ if (actualPropertyValue is string && expectedPropertyValue is not string)`
`409`	`409`	`{`
`410`	`410`	`actualPropertyValue = LanguagePrimitives.ConvertTo(actualPropertyValue, expectedPropertyValue.GetType(), CultureInfo.InvariantCulture);`
`411`	`411`	`}`
Original file line number	Diff line number	Diff line change
`@@ -667,7 +667,7 @@ internal virtual void PrepareSession()`
`667`	`667`
`668`	`668`	`// null is not valid value for header.`
`669`	`669`	`// We silently ignore header if value is null.`
`670`		`- if (!(value is null))`
	`670`	`+ if (value is not null)`
`671`	`671`	`{`
`672`	`672`	`// add the header value (or overwrite it if already present)`
`673`	`673`	`WebSession.Headers[key] = value.ToString();`
Original file line number	Diff line number	Diff line change
`@@ -525,7 +525,7 @@ private static ErrorRecord ConvertToErrorRecord(object obj)`
`525`	`525`	`if (mshobj != null)`
`526`	`526`	`{`
`527`	`527`	`object baseObject = mshobj.BaseObject;`
`528`		`- if (!(baseObject is PSCustomObject))`
	`528`	`+ if (baseObject is not PSCustomObject)`
`529`	`529`	`{`
`530`	`530`	`obj = baseObject;`
`531`	`531`	`}`
Original file line number	Diff line number	Diff line change
`@@ -938,7 +938,7 @@ public void Register()`
`938`	`938`	`{ }`
`939`	`939`	`}`
`940`	`940`
`941`		`- if (!(ex is ScheduledJobException))`
	`941`	`+ if (ex is not ScheduledJobException)`
`942`	`942`	`{`
`943`	`943`	`// Wrap in ScheduledJobException type.`
`944`	`944`	`string msg = StringUtil.Format(ScheduledJobErrorStrings.ErrorRegisteringDefinitionStore, this.Name);`
Original file line number	Diff line number	Diff line change
`@@ -783,7 +783,7 @@ protected override void ProcessRecord()`
`783`	`783`	`{`
`784`	`784`	`customDescriptor = PSObject.Base(methodInfo.Invoke());`
`785`	`785`
`786`		`- if (!(customDescriptor is FileSystemSecurity))`
	`786`	`+ if (customDescriptor is not FileSystemSecurity)`
`787`	`787`	`{`
`788`	`788`	`customDescriptor = new CommonSecurityDescriptor(false, false, customDescriptor.ToString());`
`789`	`789`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1195,7 +1195,7 @@ protected override bool ItemExists(string path)`
`1195`	`1195`	`// If the inner exception is not of that type`
`1196`	`1196`	`// then we need to rethrow`
`1197`	`1197`	`//`
`1198`		`- if (!(e.InnerException is CertificateProviderItemNotFoundException))`
	`1198`	`+ if (e.InnerException is not CertificateProviderItemNotFoundException)`
`1199`	`1199`	`{`
`1200`	`1200`	`throw;`
`1201`	`1201`	`}`