Apply feedback from review

Author: ericstj

xml/System.Runtime.ExceptionServices/ExceptionHandling.xml

@@ -43,7 +43,7 @@
         <Parameter Name="exception" Type="System.Object" />
       </Parameters>
       <Docs>
-        <param name="exception">Exception to pass to event handlers.</param>
+        <param name="exception">The exception to pass to event handlers.</param>
         <summary>Raises the <see cref="E:System.AppDomain.UnhandledException" /> event.</summary>
         <remarks>
           <para>This method will raise the <see cref="E:System.AppDomain.UnhandledException" /> event and then return.</para>
@@ -79,8 +79,8 @@
           <para>If the handler returns <see langword="false" />, the exception will continue to propagate as unhandled.</para>
           <para>The intent of this handler is to allow the user to handle unhandled exceptions gracefully when the runtime is being used in certain scenarios. Scenarios such as REPLs or game scripting that host plug-ins are not able to handle unhandled exceptions thrown by those plug-ins.</para>
         </remarks>
-        <exception cref="T:System.ArgumentNullException">If handler is null</exception>
-        <exception cref="T:System.InvalidOperationException">If a handler is already set</exception>
+        <exception cref="T:System.ArgumentNullException"><paramref name="handler" /> is <see langword="null" />.</exception>
+        <exception cref="T:System.InvalidOperationException">A handler has already been set.</exception>
       </Docs>
     </Member>
   </Members>

xml/System.Runtime.InteropServices.Java/ComponentCrossReference.xml

@@ -44,7 +44,7 @@
         <ReturnType>System.UIntPtr</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Index of the destination group.</summary>
+        <summary>Specifies the index of the destination group.</summary>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -64,7 +64,7 @@
         <ReturnType>System.UIntPtr</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Index of the source group.</summary>
+        <summary>Specifies the index of the source group.</summary>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System.Runtime.InteropServices.Java/StronglyConnectedComponent.xml

@@ -65,7 +65,7 @@
         <ReturnType>System.UIntPtr</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Number of objects in each collection.</summary>
+        <summary>Specifies the number of objects in each collection.</summary>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System.Runtime.InteropServices/CollectionsMarshal.xml

@@ -56,9 +56,9 @@
       </Parameters>
       <Docs>
         <param name="array">The <see cref="T:System.Collections.BitArray" /> whose backing storage should be viewed.</param>
-        <summary>Get a <see cref="T:System.Span`1" /> view over a <see cref="T:System.Collections.BitArray" />'s data.</summary>
+        <summary>Gets a <see cref="T:System.Span`1" /> view over a <see cref="T:System.Collections.BitArray" />'s data.</summary>
         <returns>To be added.</returns>
-        <remarks>&lt;para&gt;&lt;para&gt; The &lt;see cref="T:System.Collections.BitArray" /&gt; may have more capacity than is required to store the number of bits represented by &lt;see cref="P:System.Collections.BitArray.Length" /&gt;. The returned span's &lt;see cref="P:System.Span`1.Length" /&gt; will be the smallest number of bytes capable of representing that length. If the &lt;see cref="T:System.Collections.BitArray" /&gt;'s length is not evenly divisible by 8, the last byte of the span may contain extraneous bits that do not represent elements in the &lt;see cref="T:System.Collections.BitArray" /&gt;.&lt;/para&gt;&lt;para&gt;These may be ignored.&lt;/para&gt;&lt;para&gt;&lt;/para&gt;&lt;para&gt; The length of the &lt;see cref="T:System.Collections.BitArray" /&gt; should not be changed while the resulting &lt;see cref="T:System.Span`1" /&gt; is in use. After such a change, the span may no longer refer to the &lt;see cref="T:System.Collections.BitArray" /&gt;'s backing storage.&lt;/para&gt;&lt;para&gt;&lt;/para&gt;</remarks>
+        <remarks><para>The <see cref="T:System.Collections.BitArray" /> may have more capacity than is required to store the number of bits represented by <see cref="P:System.Collections.BitArray.Length" />.</para><para>The returned span's <see cref="P:System.Span`1.Length" /> will be the smallest number of bytes capable of representing that length. If the <see cref="T:System.Collections.BitArray" />'s length is not evenly divisible by 8, the last byte of the span may contain extraneous bits that do not represent elements in the <see cref="T:System.Collections.BitArray" />.</para><para>These may be ignored.</para><para></para><para> The length of the <see cref="T:System.Collections.BitArray" /> should not be changed while the resulting <see cref="T:System.Span`1" /> is in use. After such a change, the span may no longer refer to the <see cref="T:System.Collections.BitArray" />'s backing storage.</para></remarks>
       </Docs>
     </Member>
     <Member MemberName="AsSpan&lt;T&gt;">

xml/System.Runtime.InteropServices/ComWrappers.xml

@@ -188,15 +188,15 @@ If the object cannot be created and `null` is returned, the call to <xref:System
         <Parameter Name="wrapperFlags" Type="System.Runtime.InteropServices.CreatedWrapperFlags" RefType="out" Index="3" FrameworkAlternate="net-10.0" />
       </Parameters>
       <Docs>
-        <param name="externalComObject">Object to import for usage into the .NET runtime.</param>
-        <param name="flags">Flags used to describe the external object.</param>
-        <param name="userState">User state provided by the call to <see cref="M:System.Runtime.InteropServices.ComWrappers.GetOrCreateObjectForComInstance(System.IntPtr,System.Runtime.InteropServices.CreateObjectFlags,System.Object)" />.</param>
-        <param name="wrapperFlags">Flags used to describe the created wrapper object.</param>
-        <summary>Create a managed object for the object pointed at by <paramref name="externalComObject" /> respecting the values of <paramref name="flags" />.</summary>
+        <param name="externalComObject">The object to import for usage into the .NET runtime.</param>
+        <param name="flags">A bitwise combination of the enumeration values that describes the external object.</param>
+        <param name="userState">The user state provided by the call to <see cref="M:System.Runtime.InteropServices.ComWrappers.GetOrCreateObjectForComInstance(System.IntPtr,System.Runtime.InteropServices.CreateObjectFlags,System.Object)" />.</param>
+        <param name="wrapperFlags">When this method returns, contains the flags used to describe the created wrapper object.</param>
+        <summary>Creates a managed object for the object pointed at by <paramref name="externalComObject" /> respecting the values of <paramref name="flags" />.</summary>
         <returns>Returns a managed object associated with the supplied external COM object.</returns>
         <remarks>
           <para>The default implementation throws <see cref="T:System.NotImplementedException" />.</para>
-          <para>If the object cannot be created and <code>null</code> is returned, the call to <see cref="M:System.Runtime.InteropServices.ComWrappers.GetOrCreateObjectForComInstance(System.IntPtr,System.Runtime.InteropServices.CreateObjectFlags,System.Object)" /> will throw a <see cref="T:System.ArgumentNullException" />.</para>
+          <para>If the object can't be created and <code>null</code> is returned, the call to <see cref="M:System.Runtime.InteropServices.ComWrappers.GetOrCreateObjectForComInstance(System.IntPtr,System.Runtime.InteropServices.CreateObjectFlags,System.Object)" /> will throw an <see cref="T:System.ArgumentNullException" />.</para>
         </remarks>
       </Docs>
     </Member>
@@ -353,14 +353,13 @@ If a managed object was previously created for the specified `externalComObject`
         </Parameter>
       </Parameters>
       <Docs>
-        <param name="externalComObject">Object to import for usage into the .NET runtime.</param>
-        <param name="flags">Flags used to describe the external object.</param>
+        <param name="externalComObject">The object to import for usage into the .NET runtime.</param>
+        <param name="flags">A bitwise combination of the enumeration values that describes the external object.</param>
         <param name="userState">A state object to use to help create the wrapping .NET object.</param>
-        <summary>Get the currently registered managed object or creates a new managed object and registers it.</summary>
+        <summary>Gets the currently registered managed object or creates a new managed object and registers it.</summary>
         <returns>Returns a managed object associated with the supplied external COM object.</returns>
         <remarks>
-          <para>If a managed object was previously created for the specified <paramref name="externalComObject" /> using this <see cref="T:System.Runtime.InteropServices.ComWrappers" /> instance, the previously created object will be returned.</para>
-          <para>If not, a new one will be created.</para>
+          If a managed object was previously created for the specified <paramref name="externalComObject" /> using this <see cref="T:System.Runtime.InteropServices.ComWrappers" /> instance, the previously created object is returned. If not, a new one is created.
         </remarks>
       </Docs>
     </Member>

xml/System.Runtime.InteropServices/CreatedWrapperFlags.xml

@@ -19,7 +19,7 @@
     </Attribute>
   </Attributes>
   <Docs>
-    <summary>Enumeration of flags for <see cref="M:System.Runtime.InteropServices.ComWrappers.CreateObject(System.IntPtr,System.Runtime.InteropServices.CreateObjectFlags,System.Object,System.Runtime.InteropServices.CreatedWrapperFlags@)" />.</summary>
+    <summary>Specifies flags for <see cref="M:System.Runtime.InteropServices.ComWrappers.CreateObject(System.IntPtr,System.Runtime.InteropServices.CreateObjectFlags,System.Object,System.Runtime.InteropServices.CreatedWrapperFlags@)" />.</summary>
     <remarks>To be added.</remarks>
   </Docs>
   <Members>
@@ -60,7 +60,7 @@
       </ReturnValue>
       <MemberValue>2</MemberValue>
       <Docs>
-        <summary>The managed object doesn't keep the native object alive. It represents an equivalent value.</summary>
+        <summary>A managed object that doesn't keep the native object alive. The managed object represents an equivalent value.</summary>
       </Docs>
     </Member>
     <Member MemberName="TrackerObject">
@@ -80,7 +80,7 @@
       </ReturnValue>
       <MemberValue>1</MemberValue>
       <Docs>
-        <summary>Indicate if the supplied external COM object implements the <see href="https://learn.microsoft.com/windows/win32/api/windows.ui.xaml.hosting.referencetracker/nn-windows-ui-xaml-hosting-referencetracker-ireferencetracker">IReferenceTracker</see>.</summary>
+        <summary>The supplied external COM object implements the <see href="https://learn.microsoft.com/windows/win32/api/windows.ui.xaml.hosting.referencetracker/nn-windows-ui-xaml-hosting-referencetracker-ireferencetracker">IReferenceTracker</see>.</summary>
       </Docs>
     </Member>
   </Members>

xml/System.Runtime.InteropServices/GCHandleExtensions.xml

@@ -59,13 +59,13 @@
       </Parameters>
       <Docs>
         <typeparam name="T">To be added.</typeparam>
-        <param name="handle">The handle to retrieve pointer from.</param>
-        <summary>Retrieves the address of array data in <paramref name="handle" />.</summary>
+        <param name="handle">The handle to retrieve the pointer from.</param>
+        <summary>Retrieves the address of the array data in <paramref name="handle" />.</summary>
         <returns>
-          <para>The address of 0th array element the pinned array, or <see langword="null" /> if the handle doesn't point to any object.</para>
+          <para>The address of the 0th array element of the pinned array, or <see langword="null" /> if the handle doesn't point to any object.</para>
         </returns>
         <remarks>To be added.</remarks>
-        <exception cref="T:System.NullReferenceException">If the handle is not initialized or already disposed.</exception>
+        <exception cref="T:System.NullReferenceException">The handle is not initialized or is already disposed.</exception>
       </Docs>
     </Member>
     <Member MemberName="GetAddressOfStringData">
@@ -92,13 +92,13 @@
         <Parameter Name="handle" Type="System.Runtime.InteropServices.PinnedGCHandle&lt;System.String&gt;" RefType="this" />
       </Parameters>
       <Docs>
-        <param name="handle">The handle to retrieve pointer from.</param>
-        <summary>Retrieves the address string data in <paramref name="handle" />.</summary>
+        <param name="handle">The handle to retrieve the pointer from.</param>
+        <summary>Retrieves the address of the string data in <paramref name="handle" />.</summary>
         <returns>
-          <para>The address of 0th character of the pinned <see cref="T:System.String" />, or <see langword="null" /> if the handle doesn't point to any object.</para>
+          <para>The address of the 0th character of the pinned <see cref="T:System.String" />, or <see langword="null" /> if the handle doesn't point to any object.</para>
         </returns>
         <remarks>To be added.</remarks>
-        <exception cref="T:System.NullReferenceException">If the handle is not initialized or already disposed.</exception>
+        <exception cref="T:System.NullReferenceException">The handle is not initialized or is already disposed.</exception>
       </Docs>
     </Member>
   </Members>

xml/System.Runtime.InteropServices/GCHandle`1.xml

@@ -41,11 +41,8 @@
   </Attributes>
   <Docs>
     <typeparam name="T">The type of the object this <see cref="T:System.Runtime.InteropServices.GCHandle`1" /> tracks to.</typeparam>
-    <summary>
-      <para>Represents a strongly-typed GC handle to a managed object.</para>
-      <para>A GC handle is used to work with object references in unmanaged code.</para>
-    </summary>
-    <remarks>&lt;para&gt;&lt;para&gt;This type corresponds to &lt;see cref="F:System.Runtime.InteropServices.GCHandleType.Normal" /&gt;.&lt;/para&gt;&lt;para&gt; This type is unsafe if used incorrectly. Incorrect usage like mismanagement of lifetime, providing invalid handle value or concurrent disposal can result in hard to diagnose crashes or data corruptions.&lt;/para&gt;&lt;para&gt;&lt;/para&gt;</remarks>
+    <summary>Represents a strongly typed GC handle to a managed object.</summary>
+    <remarks><para>A GC handle is used to work with object references in unmanaged code.</para><para>This type corresponds to <see cref="F:System.Runtime.InteropServices.GCHandleType.Normal" />.</para><para>This type is unsafe if used incorrectly. Incorrect usage like mismanagement of lifetime, providing invalid handle value, or concurrent disposal can result in hard to diagnose crashes or data corruptions.</para></remarks>
     <seealso cref="T:System.Runtime.InteropServices.GCHandle" />
   </Docs>
   <Members>
@@ -203,7 +200,7 @@
         <param name="value">An <see cref="T:System.IntPtr" /> handle to a managed object to create a <see cref="T:System.Runtime.InteropServices.GCHandle`1" /> object from.</param>
         <summary>Returns a new <see cref="T:System.Runtime.InteropServices.GCHandle`1" /> object created from a handle to a managed object.</summary>
         <returns>A new <see cref="T:System.Runtime.InteropServices.GCHandle`1" /> object that corresponds to the value parameter.</returns>
-        <remarks>&lt;para&gt;&lt;para&gt;This method doesn't validate the provided handle value. The caller must ensure the validity of the handle.&lt;/para&gt;&lt;para&gt; The &lt;see cref="T:System.IntPtr" /&gt; representation of &lt;see cref="T:System.Runtime.InteropServices.GCHandle`1" /&gt; is not interchangable with &lt;see cref="T:System.Runtime.InteropServices.GCHandle" /&gt;.&lt;/para&gt;&lt;para&gt;&lt;/para&gt;</remarks>
+        <remarks><para><para>This method doesn't validate the provided handle value. The caller must ensure the validity of the handle.</para><para> The <see cref="T:System.IntPtr" /> representation of <see cref="T:System.Runtime.InteropServices.GCHandle`1" /> is not interchangable with <see cref="T:System.Runtime.InteropServices.GCHandle" />.</para><para></para></remarks>
       </Docs>
     </Member>
     <Member MemberName="GetHashCode">
@@ -256,7 +253,7 @@
         <ReturnType>System.Boolean</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Determine whether this handle has been allocated or not.</summary>
+        <summary>Gets a value that indicates whether this handle has been allocated or not.</summary>
         <value>To be added.</value>
         <remarks>To be added.</remarks>
       </Docs>
@@ -290,7 +287,7 @@
         <summary>Gets or sets the object this handle represents.</summary>
         <value>To be added.</value>
         <remarks>To be added.</remarks>
-        <exception cref="T:System.NullReferenceException">If the handle is not initialized or already disposed.</exception>
+        <exception cref="T:System.NullReferenceException">The handle is not initialized or already disposed.</exception>  
       </Docs>
     </Member>
     <Member MemberName="ToIntPtr">