Fix that/which grammar (#11141)

Author: gewarren

xml/System.Collections.Specialized/HybridDictionary.xml

@@ -71,7 +71,7 @@
       <format type="text/markdown"><![CDATA[
 
 ## Remarks
- This class is recommended for cases where the number of elements in a dictionary is unknown. It takes advantage of the improved performance of a <xref:System.Collections.Specialized.ListDictionary> with small collections, and offers the flexibility of switching to a <xref:System.Collections.Hashtable> which handles larger collections better than <xref:System.Collections.Specialized.ListDictionary>.
+ This class is recommended for cases where the number of elements in a dictionary is unknown. It takes advantage of the improved performance of a <xref:System.Collections.Specialized.ListDictionary> with small collections, and offers the flexibility of switching to a <xref:System.Collections.Hashtable>, which handles larger collections better than <xref:System.Collections.Specialized.ListDictionary>.
 
  If the initial size of the collection is greater than the optimal size for a <xref:System.Collections.Specialized.ListDictionary>, the collection is stored in a <xref:System.Collections.Hashtable> to avoid the overhead of copying elements from the <xref:System.Collections.Specialized.ListDictionary> to a <xref:System.Collections.Hashtable>.
 

xml/System.Collections.Specialized/NameObjectCollectionBase.xml

@@ -94,9 +94,7 @@
  `null` is allowed as a key or as a value.
 
 > [!CAUTION]
->  The <xref:System.Collections.Specialized.NameObjectCollectionBase.BaseGet%2A> method does not distinguish between `null` which is returned because the specified key is not found and `null` which is returned because the value associated with the key is `null`.
-
-
+> The <xref:System.Collections.Specialized.NameObjectCollectionBase.BaseGet%2A> method does not distinguish between `null` that's returned because the specified key is not found and `null` that's returned because the value associated with the key is `null`.
 
 ## Examples
  The following code example shows how to implement and use the <xref:System.Collections.Specialized.NameObjectCollectionBase> class.

xml/System.Collections.Specialized/NameValueCollection.xml

@@ -76,9 +76,7 @@
  `null` is allowed as a key or as a value.
 
 > [!CAUTION]
->  The <xref:System.Collections.Specialized.NameValueCollection.Get%2A> method does not distinguish between `null` which is returned because the specified key is not found and `null` which is returned because the value associated with the key is `null`.
-
-
+> The <xref:System.Collections.Specialized.NameValueCollection.Get%2A> method does not distinguish between `null` that's returned because the specified key is not found and `null` that's returned because the value associated with the key is `null`.
 
 ## Examples
  :::code language="cpp" source="~/snippets/cpp/VS_Snippets_CLR_System/system.Collections.Specialized.NameValueCollection2/CPP/nvc.cpp" id="Snippet1":::

xml/System.Collections/ArrayList.xml

@@ -1911,8 +1911,8 @@ This method is an `O(1)` operation.
       <Docs>
         <param name="index">The zero-based <see cref="T:System.Collections.ArrayList" /> index at which the range starts.</param>
         <param name="count">The number of elements in the range.</param>
-        <summary>Returns an <see cref="T:System.Collections.ArrayList" /> which represents a subset of the elements in the source <see cref="T:System.Collections.ArrayList" />.</summary>
-        <returns>An <see cref="T:System.Collections.ArrayList" /> which represents a subset of the elements in the source <see cref="T:System.Collections.ArrayList" />.</returns>
+        <summary>Returns an <see cref="T:System.Collections.ArrayList" /> that represents a subset of the elements in the source <see cref="T:System.Collections.ArrayList" />.</summary>
+        <returns>An <see cref="T:System.Collections.ArrayList" /> that represents a subset of the elements in the source <see cref="T:System.Collections.ArrayList" />.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[
 

xml/System.Collections/Hashtable.xml

@@ -652,7 +652,7 @@ Each element is a key/value pair stored in a <xref:System.Collections.Dictionary
       </Parameters>
       <Docs>
         <param name="d">The <see cref="T:System.Collections.IDictionary" /> object to copy to a new <see cref="T:System.Collections.Hashtable" /> object.</param>
-        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value which provides the best performance. The result is the maximum ratio of elements to buckets.</param>
+        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value that provides the best performance. The result is the maximum ratio of elements to buckets.</param>
         <summary>Initializes a new instance of the <see cref="T:System.Collections.Hashtable" /> class by copying the elements from the specified dictionary to the new <see cref="T:System.Collections.Hashtable" /> object. The new <see cref="T:System.Collections.Hashtable" /> object has an initial capacity equal to the number of elements copied, and uses the specified load factor, and the default hash code provider and comparer.</summary>
         <remarks>
           <format type="text/markdown"><![CDATA[
@@ -934,7 +934,7 @@ Each element is a key/value pair stored in a <xref:System.Collections.Dictionary
       </Parameters>
       <Docs>
         <param name="capacity">The approximate number of elements that the <see cref="T:System.Collections.Hashtable" /> object can initially contain.</param>
-        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value which provides the best performance. The result is the maximum ratio of elements to buckets.</param>
+        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value that provides the best performance. The result is the maximum ratio of elements to buckets.</param>
         <summary>Initializes a new, empty instance of the <see cref="T:System.Collections.Hashtable" /> class using the specified initial capacity and load factor, and the default hash code provider and comparer.</summary>
         <remarks>
           <format type="text/markdown"><![CDATA[
@@ -1233,7 +1233,7 @@ Each element is a key/value pair stored in a <xref:System.Collections.Dictionary
       </Parameters>
       <Docs>
         <param name="d">The <see cref="T:System.Collections.IDictionary" /> object to copy to a new <see cref="T:System.Collections.Hashtable" /> object.</param>
-        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value which provides the best performance. The result is the maximum ratio of elements to buckets.</param>
+        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value that provides the best performance. The result is the maximum ratio of elements to buckets.</param>
         <param name="equalityComparer">The <see cref="T:System.Collections.IEqualityComparer" /> object that defines the hash code provider and the comparer to use with the <see cref="T:System.Collections.Hashtable" />.
 
  -or-
@@ -1435,7 +1435,7 @@ Each element is a key/value pair stored in a <xref:System.Collections.Dictionary
       </Parameters>
       <Docs>
         <param name="capacity">The approximate number of elements that the <see cref="T:System.Collections.Hashtable" /> object can initially contain.</param>
-        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value which provides the best performance. The result is the maximum ratio of elements to buckets.</param>
+        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value that provides the best performance. The result is the maximum ratio of elements to buckets.</param>
         <param name="equalityComparer">The <see cref="T:System.Collections.IEqualityComparer" /> object that defines the hash code provider and the comparer to use with the <see cref="T:System.Collections.Hashtable" />.
 
  -or-
@@ -1550,7 +1550,7 @@ Each element is a key/value pair stored in a <xref:System.Collections.Dictionary
       </Parameters>
       <Docs>
         <param name="d">The <see cref="T:System.Collections.IDictionary" /> object to copy to a new <see cref="T:System.Collections.Hashtable" /> object.</param>
-        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value which provides the best performance. The result is the maximum ratio of elements to buckets.</param>
+        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value that provides the best performance. The result is the maximum ratio of elements to buckets.</param>
         <param name="hcp">The <see cref="T:System.Collections.IHashCodeProvider" /> object that supplies the hash codes for all keys in the <see cref="T:System.Collections.Hashtable" />.
 
  -or-
@@ -1660,7 +1660,7 @@ Each element is a key/value pair stored in a <xref:System.Collections.Dictionary
       </Parameters>
       <Docs>
         <param name="capacity">The approximate number of elements that the <see cref="T:System.Collections.Hashtable" /> object can initially contain.</param>
-        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value which provides the best performance. The result is the maximum ratio of elements to buckets.</param>
+        <param name="loadFactor">A number in the range from 0.1 through 1.0 that is multiplied by the default value that provides the best performance. The result is the maximum ratio of elements to buckets.</param>
         <param name="hcp">The <see cref="T:System.Collections.IHashCodeProvider" /> object that supplies the hash codes for all keys in the <see cref="T:System.Collections.Hashtable" />.
 
  -or-

xml/System.Diagnostics.Metrics/MeterListener.xml

@@ -274,8 +274,8 @@
       </Parameters>
       <Docs>
         <typeparam name="T">The type of the numeric measurement.</typeparam>
-        <param name="measurementCallback">The callback which can be used to get measurement recording of numeric type T.</param>
-        <summary>Sets a callback for a specific numeric type to get the measurement recording notification from all instruments which enabled listening and was created with the same specified numeric type.
+        <param name="measurementCallback">The callback that can be used to get measurement recording of numeric type T.</param>
+        <summary>Sets a callback for a specific numeric type to get the measurement recording notification from all instruments that enabled listening and were created with the same specified numeric type.
             If a measurement of type T is recorded and a callback of type T is registered, that callback will be used.</summary>
         <remarks>To be added.</remarks>
       </Docs>

xml/System.Diagnostics/ActivityCreationOptions`1.xml

@@ -67,7 +67,7 @@ The possible generic type parameters are <see cref="T:System.Diagnostics.Activit
         <ReturnType>System.Diagnostics.ActivityKind</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Gets the activity kind which the activity will be created with.</summary>
+        <summary>Gets the activity kind that the activity will be created with.</summary>
         <value>One of the enumeration values that represent an activity kind.</value>
         <remarks>To be added.</remarks>
       </Docs>

xml/System.Diagnostics/DiagnosticSource.xml

@@ -144,8 +144,8 @@ If there is expensive setup for the notification, you can call IsEnabled before
       </Parameters>
       <Docs>
         <param name="name">The name of the event being written.</param>
-        <param name="arg1">An object that represents the additional context for IsEnabled. Consumers should expect to receive <see langword="null" /> which may indicate that producer called pure IsEnabled(string) to check if consumer wants to get notifications for such events at all. Based on that, producer may call IsEnabled(string, object, object) again with non-<see langword="null" /> context.</param>
-        <param name="arg2">Optional. An object that represents the additional context for IsEnabled. <see langword="null" /> by default. Consumers should expect to receive <see langword="null" /> which may indicate that producer called pure IsEnabled(string) or producer passed all necessary context in <paramref name="arg1" />.</param>
+        <param name="arg1">An object that represents the additional context for IsEnabled. Consumers should expect to receive <see langword="null" />, which might indicate that producer called pure IsEnabled(string) to check if consumer wants to get notifications for such events at all. Based on that, producer may call IsEnabled(string, object, object) again with non-<see langword="null" /> context.</param>
+        <param name="arg2">Optional. An object that represents the additional context for IsEnabled. <see langword="null" /> by default. Consumers should expect to receive <see langword="null" />, which might indicate that producer called pure IsEnabled(string) or producer passed all necessary context in <paramref name="arg1" />.</param>
         <summary>Verifies it the notification event is enabled.</summary>
         <returns>
           <see langword="true" /> if the notification event is enabled, <see langword="false" /> otherwise.</returns>
@@ -546,7 +546,7 @@ Consumers can access <xref:System.Diagnostics.Activity.Current?displayProperty=n
       </Parameters>
       <Docs>
         <param name="name">The name of the event being written.</param>
-        <param name="value">An object that represents the value being passed as a payload for the event. This is often an anonymous type which contains several sub-values.</param>
+        <param name="value">An object that represents the value being passed as a payload for the event. This is often an anonymous type that contains several subvalues.</param>
         <summary>Provides a generic way of logging complex payloads.
 </summary>
         <remarks>