[StyleCleanUp] Address IDE Warnings Part 1 (#10168)

Fixes IDE0017: Use object initializers in src

Author: harshit7962

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

@@ -185,9 +185,6 @@ dotnet_diagnostic.CA5362.severity = suggestion
 # IDE0005: Using directive is unnecessary.
 dotnet_diagnostic.IDE0005.severity = suggestion
 
-# IDE0017: Simplify object initialization
-dotnet_diagnostic.IDE0017.severity = suggestion
-
 # IDE0030: Null check can be simplified
 dotnet_diagnostic.IDE0030.severity = suggestion
 

src/Microsoft.DotNet.Wpf/src/Extensions/PresentationFramework-SystemData/SystemDataExtension.cs

@@ -49,16 +49,22 @@ internal override bool IsDataSetCollectionProperty(PropertyDescriptor pd)
             {
                 // lazy load the types for the offending PD's.  They're internal, so
                 // we get them indirectly.
-                DataSet dataset = new DataSet();
-                dataset.Locale = System.Globalization.CultureInfo.InvariantCulture;
+                DataSet dataset = new DataSet
+                {
+                    Locale = System.Globalization.CultureInfo.InvariantCulture
+                };
 
-                DataTable table1 = new DataTable("Table1");
-                table1.Locale = System.Globalization.CultureInfo.InvariantCulture;
+                DataTable table1 = new DataTable("Table1")
+                {
+                    Locale = System.Globalization.CultureInfo.InvariantCulture
+                };
                 table1.Columns.Add("ID", typeof(int));
                 dataset.Tables.Add(table1);
 
-                DataTable table2 = new DataTable("Table2");
-                table2.Locale = System.Globalization.CultureInfo.InvariantCulture;
+                DataTable table2 = new DataTable("Table2")
+                {
+                    Locale = System.Globalization.CultureInfo.InvariantCulture
+                };
                 table2.Columns.Add("ID", typeof(int));
                 dataset.Tables.Add(table2);
 

src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/MS/Internal/MarkupCompiler/MarkupCompiler.cs

@@ -762,9 +762,10 @@ public override void WritePIMapping(XamlPIMappingNode xamlPIMappingNode)
                 if (addMapping)
                 {
                     ClrNamespaceAssemblyPair namespaceMapping = new ClrNamespaceAssemblyPair(xamlPIMappingNode.ClrNamespace,
-                                                                                             xamlPIMappingNode.AssemblyName);
-
-                    namespaceMapping.LocalAssembly = true;
+                                                                                             xamlPIMappingNode.AssemblyName)
+                    {
+                        LocalAssembly = true
+                    };
                     XamlTypeMapper.PITable[xamlPIMappingNode.XmlNamespace] = namespaceMapping;
                     XamlTypeMapper.InvalidateMappingCache(xamlPIMappingNode.XmlNamespace);
                     if (!_pass2 && BamlRecordWriter != null)

src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/MS/Internal/MarkupCompiler/ParserExtension.cs

@@ -223,12 +223,14 @@ internal bool DoCompilation(string assemblyName, string language, string rootNam
         {
             bool ret = true;
 
-            CompilationUnit compUnit = new CompilationUnit(assemblyName, language, rootNamespace, fileList);
-            compUnit.Pass2 = isSecondPass;
+            CompilationUnit compUnit = new CompilationUnit(assemblyName, language, rootNamespace, fileList)
+            {
+                Pass2 = isSecondPass,
 
-            // Set some properties required by the CompilationUnit
-            compUnit.ApplicationFile = _applicationMarkup;
-            compUnit.SourcePath = _sourceDir;
+                // Set some properties required by the CompilationUnit
+                ApplicationFile = _applicationMarkup,
+                SourcePath = _sourceDir
+            };
 
             //Set the properties required by MarkupCompiler
 

src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/MS/Internal/Tasks/CompilerWrapper.cs

@@ -1472,9 +1472,11 @@ private void GenerateOutputItemsForCompiledXamlFiles(ITaskItem[] inputXamlItemLi
                 {
                     TaskItem codeItem;
 
-                    codeItem = new TaskItem();
-                    codeItem.ItemSpec = genLangFilePath;
-                    
+                    codeItem = new TaskItem
+                    {
+                        ItemSpec = genLangFilePath
+                    };
+
                     outputCodeFileList.Add(codeItem);
 
                     Log.LogMessageFromResources(MessageImportance.Low, nameof(SR.GeneratedCodeFile), codeItem.ItemSpec);
@@ -1577,8 +1579,10 @@ private TaskItem GenerateBamlItem(string bamlFile, ITaskItem SourceItem)
         {
             TaskItem bamlItem;
 
-            bamlItem =  new TaskItem();
-            bamlItem.ItemSpec = bamlFile;
+            bamlItem = new TaskItem
+            {
+                ItemSpec = bamlFile
+            };
 
             //
             // Transfer some special custom attributes from source task item

src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/Microsoft/Build/Tasks/Windows/MarkupCompilePass1.cs

@@ -790,8 +790,10 @@ private TaskItem GenerateBamlItem(string resolvedXamlfile, bool localizable, str
                 // Generate a TaskItem for it.
                 //
 
-                bamlItem = new TaskItem();
-                bamlItem.ItemSpec = bamlFile;
+                bamlItem = new TaskItem
+                {
+                    ItemSpec = bamlFile
+                };
 
                 // Transfer the metadata value from source item to the generated baml item.
                 bamlItem.SetMetadata(SharedStrings.Localizable, localizable ?  "True" : "False");

src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/Microsoft/Build/Tasks/Windows/MarkupCompilePass2.cs

@@ -117,9 +117,11 @@ public override bool Execute()
                 // Update the manifest file.
                 try
                 {
-                    manifestWriter = new XmlTextWriter(appManifestFile, System.Text.Encoding.UTF8);
-                    manifestWriter.Formatting = Formatting.Indented;
-                    manifestWriter.Indentation = 4;
+                    manifestWriter = new XmlTextWriter(appManifestFile, System.Text.Encoding.UTF8)
+                    {
+                        Formatting = Formatting.Indented,
+                        Indentation = 4
+                    };
                     manifestDocument.WriteTo(manifestWriter);
                 }
                 finally

src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/Microsoft/Build/Tasks/Windows/UpdateManifestForBrowserApplication.cs

@@ -1,4 +1,4 @@
-// Licensed to the .NET Foundation under one or more agreements.
+// Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
@@ -114,15 +114,16 @@ internal static Quaternion InterpolateQuaternion(Quaternion from, Quaternion to,
 
         internal static Rect InterpolateRect(Rect from, Rect to, Double progress)
         {
-            Rect temp = new Rect();
-
-            // from + ((from - to) * progress)
-            temp.Location = new Point(
+            Rect temp = new Rect
+            {
+                // from + ((from - to) * progress)
+                Location = new Point(
                 from.Location.X + ((to.Location.X - from.Location.X) * progress),
-                from.Location.Y + ((to.Location.Y - from.Location.Y) * progress));
-            temp.Size = new Size(
+                from.Location.Y + ((to.Location.Y - from.Location.Y) * progress)),
+                Size = new Size(
                 from.Size.Width + ((to.Size.Width - from.Size.Width) * progress),
-                from.Size.Height + ((to.Size.Height - from.Size.Height) * progress));
+                from.Size.Height + ((to.Size.Height - from.Size.Height) * progress))
+            };
 
             return temp;
         }
@@ -563,14 +564,15 @@ internal static Quaternion ScaleQuaternion(Quaternion value, Double factor)
 
         internal static Rect ScaleRect(Rect value, Double factor)
         {
-            Rect temp = new Rect();
-
-            temp.Location = new Point(
+            Rect temp = new Rect
+            {
+                Location = new Point(
                 value.Location.X * factor,
-                value.Location.Y * factor);
-            temp.Size = new Size(
+                value.Location.Y * factor),
+                Size = new Size(
                 value.Size.Width * factor,
-                value.Size.Height * factor);
+                value.Size.Height * factor)
+            };
 
             return temp;
         }

src/Microsoft.DotNet.Wpf/src/PresentationCore/MS/internal/AnimatedTypeHelpers.cs

@@ -1,4 +1,4 @@
-// Licensed to the .NET Foundation under one or more agreements.
+// Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
@@ -221,12 +221,13 @@ private CompositeFontParser(Stream fileStream)
         /// </summary>
         private XmlReader CreateXmlReader(Stream fileStream)
         {
-            XmlReaderSettings settings = new XmlReaderSettings();
-
-            settings.CloseInput = true;
-            settings.IgnoreComments = true;
-            settings.IgnoreWhitespace = false;
-            settings.ProhibitDtd = true;
+            XmlReaderSettings settings = new XmlReaderSettings
+            {
+                CloseInput = true,
+                IgnoreComments = true,
+                IgnoreWhitespace = false,
+                ProhibitDtd = true
+            };
 
             XmlReader baseReader = XmlReader.Create(fileStream, settings);