[StyleCleanUp] Addressing IDE warnings (#10185)

* Addressing IDE0150 .editorconfig issues

* Reverting encoding changes

Author: siagupta0202

src/Microsoft.DotNet.Wpf/src/.editorconfig

@@ -257,9 +257,6 @@ dotnet_diagnostic.IDE0082.severity = suggestion
 # IDE0100: Remove redundant equality
 dotnet_diagnostic.IDE0100.severity = suggestion
 
-# IDE0150: Prefer 'null' check over type check
-dotnet_diagnostic.IDE0150.severity = suggestion
-
 # IDE0180: Use tuple to swap values
 dotnet_diagnostic.IDE0180.severity = suggestion
 

src/Microsoft.DotNet.Wpf/src/PresentationCore/MS/internal/Ink/Renderer.cs

@@ -529,7 +529,7 @@ private void AttachVisual(StrokeVisual visual, bool buildingStrokeCollection)
             {
                 // Find or create a container visual for highlighter strokes of the color
                 ContainerVisual parent = GetContainerVisual(visual.Stroke.DrawingAttributes);
-                Debug.Assert(visual is StrokeVisual);
+                Debug.Assert(visual is not null);
 
                 //insert StrokeVisuals under any non-StrokeVisuals used for dynamic inking
                 int i = 0;

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/PtsHost/FlowDocumentPage.cs

@@ -924,7 +924,7 @@ private TextPointer FindElementPosition(IInputElement e, bool isLimitedToTextVie
             Debug.Assert(e != null);
 
             // Validate that this function is only called when a TextContainer exists as complex content
-            Debug.Assert(_structuralCache.TextContainer is TextContainer);
+            Debug.Assert(_structuralCache.TextContainer is not null);
 
             TextPointer elementPosition = null;
 
@@ -941,8 +941,8 @@ private TextPointer FindElementPosition(IInputElement e, bool isLimitedToTextVie
             else
             {
                 // Else: search for e in the complex content
-                if (!(_structuralCache.TextContainer.Start is TextPointer) ||
-                    !(_structuralCache.TextContainer.End is TextPointer))
+                if (!(_structuralCache.TextContainer.Start is not null) ||
+                    !(_structuralCache.TextContainer.End is not null))
                 {
                     // Invalid TextContainer, don't search
                     return null;

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/WindowsRuntime/Generated/WinRT/ComWrappersSupport.cs

@@ -61,15 +61,15 @@ public static bool TryUnwrapObject(object o, out IObjectReference objRef)
 
             Type type = o.GetType();
             ObjectReferenceWrapperAttribute objRefWrapper = type.GetCustomAttribute<ObjectReferenceWrapperAttribute>();
-            if (objRefWrapper is object)
+            if (objRefWrapper is not null)
             {
                 objRef = (IObjectReference)type.GetField(objRefWrapper.ObjectReferenceField, BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.DeclaredOnly).GetValue(o);
                 return true;
             }
 
             ProjectedRuntimeClassAttribute projectedClass = type.GetCustomAttribute<ProjectedRuntimeClassAttribute>();
 
-            if (projectedClass is object)
+            if (projectedClass is not null)
             {
                 return TryUnwrapObject(
                     type.GetProperty(projectedClass.DefaultInterfaceProperty, BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance | BindingFlags.DeclaredOnly).GetValue(o),

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/WindowsRuntime/Generated/WinRT/ExceptionHelpers.cs

@@ -66,7 +66,7 @@ public static void ThrowExceptionForHR(int hr)
             {
                 ExceptionDispatchInfo.Capture(ex).Throw();
             }
-            else if (ex is object)
+            else if (ex is not null)
             {
                 throw ex;
             }
@@ -107,7 +107,7 @@ private static Exception GetExceptionForHR(int hr, bool useGlobalErrorState, out
                     {
                         ILanguageExceptionErrorInfo languageErrorInfo = new ABI.WinRT.Interop.ILanguageExceptionErrorInfo(languageErrorInfoRef);
                         using IObjectReference languageException = languageErrorInfo.GetLanguageException();
-                        if (languageException is object)
+                        if (languageException is not null)
                         {
                             if (languageException.IsReferenceToManagedObject)
                             {

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/WindowsRuntime/Generated/WinRT/Projections.cs

@@ -130,7 +130,7 @@ private static bool IsTypeWindowsRuntimeTypeNoArray(Type type)
                 || type == typeof(string)
                 || type == typeof(Guid)
                 || type == typeof(object)
-                || type.GetCustomAttribute<WindowsRuntimeTypeAttribute>() is object;
+                || type.GetCustomAttribute<WindowsRuntimeTypeAttribute>() is not null;
         }
 
         public static bool TryGetCompatibleWindowsRuntimeTypeForVariantType(Type type, out Type compatibleType)
@@ -205,7 +205,7 @@ internal static bool TryGetMarshalerTypeForProjectedRuntimeClass(IObjectReferenc
                 {
                     IInspectable inspectable = inspectablePtr;
                     string runtimeClassName = inspectable.GetRuntimeClassName(true);
-                    if (runtimeClassName is object)
+                    if (runtimeClassName is not null)
                     {
                         if (ProjectedRuntimeClassNames.Contains(runtimeClassName))
                         {

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/WindowsRuntime/Generated/WinRT/TypeExtensions.cs

@@ -15,7 +15,7 @@ public static Type FindHelperType(this Type type)
                 type = typeof(Exception);
             }
             Type customMapping = Projections.FindCustomHelperTypeMapping(type);
-            if (customMapping is object)
+            if (customMapping is not null)
             {
                 return customMapping;
             }
@@ -31,7 +31,7 @@ public static Type FindHelperType(this Type type)
         public static Type GetHelperType(this Type type)
         {
             var helperType = type.FindHelperType();
-            if (helperType is object)
+            if (helperType is not null)
                 return helperType;
             throw new InvalidOperationException($"Target type is not a projected type: {type.FullName}.");
         }
@@ -48,7 +48,7 @@ public static Type FindVftblType(this Type helperType)
             {
                 return null;
             }
-            if (helperType.IsGenericType && vftblType is object)
+            if (helperType.IsGenericType && vftblType is not null)
             {
                 vftblType = vftblType.MakeGenericType(helperType.GetGenericArguments());
             }

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/documents/UndoManager.cs

@@ -96,7 +96,7 @@ internal static void AttachUndoManager(DependencyObject scope, UndoManager undoM
             ArgumentNullException.ThrowIfNull(scope);
             ArgumentNullException.ThrowIfNull(undoManager);
 
-            if (undoManager is UndoManager && ((UndoManager)undoManager)._scope != null)
+            if (undoManager is not null && ((UndoManager)undoManager)._scope != null)
             {
                 throw new InvalidOperationException(SR.UndoManagerAlreadyAttached);
             }
@@ -106,7 +106,7 @@ internal static void AttachUndoManager(DependencyObject scope, UndoManager undoM
 
             // Attach the service to the scope via private dependency property
             scope.SetValue(UndoManager.UndoManagerInstanceProperty, undoManager);
-            if (undoManager is UndoManager)
+            if (undoManager is not null)
             {
                 Debug.Assert(((UndoManager)undoManager)._scope == null);
                 ((UndoManager)undoManager)._scope = scope;
@@ -141,7 +141,7 @@ internal static void DetachUndoManager(DependencyObject scope)
                 scope.ClearValue(UndoManager.UndoManagerInstanceProperty);
 
                 // Break the linkage to its scope
-                if (undoManager is UndoManager)
+                if (undoManager is not null)
                 {
                     Debug.Assert(((UndoManager)undoManager)._scope == scope);
                     ((UndoManager)undoManager)._scope = null;

src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/Automation/Peers/ItemsControlAutomationPeer.cs

@@ -424,7 +424,7 @@ internal RecyclableWrapper GetRecyclableWrapperPeer(object item)
             return _recyclableWrapperCache;
         }
 
-        // UpdateChildrenIntenal is called with ItemsInvalidateLimit to ensure we don’t fire unnecessary structure change events when items are just scrolled in/out of view in case of
+        // UpdateChildrenIntenal is called with ItemsInvalidateLimit to ensure we don’t fire unnecessary structure change events when items are just scrolled in/out of view in case of
         // virtualized controls.
         override internal IDisposable UpdateChildren()
         {
@@ -625,7 +625,7 @@ public T this[object item]
                     if (_hashtable == null)
                         _hashtable = new WeakDictionary<object,T>();
 
-                    if(!_hashtable.ContainsKey(item) && value is T)
+                    if(!_hashtable.ContainsKey(item) && value is not null)
                         _hashtable[item] = value;
                     else
                         Debug.Assert(false,"it must not add already present Item");
@@ -634,7 +634,7 @@ public T this[object item]
                 {
                     if (_list == null)
                         _list = new List<KeyValuePair<object, T>>();
-                    if(value is T)
+                    if(value is not null)
                         _list.Add(new KeyValuePair<object, T>(item, value));
                 }
 

src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/Documents/DocumentSequenceTextPointer.cs

@@ -841,7 +841,7 @@ internal static uint GetDebugId()
 
         internal static string ToString(DocumentSequenceTextPointer thisTp)
         {
-            return $"{(thisTp is DocumentSequenceTextPointer ? "DSTP" : "DSTN")} Id={thisTp.DebugId} B={thisTp.ChildBlock.DebugId} G={thisTp.ChildPointer.LogicalDirection}";
+            return $"{(thisTp is not null ? "DSTP" : "DSTN")} Id={thisTp.DebugId} B={thisTp.ChildBlock.DebugId} G={thisTp.ChildPointer.LogicalDirection}";
         }
 #endif
         #endregion Internal Methods