Apply suggestions from code review

Author: gewarren

xml/System.Formats.Cbor/CborReader.xml

@@ -104,7 +104,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets the conformance mode used by this reader.</summary>
-        <value>One of the enumeration values that represent the conformance mode used by this reader.</value>
+        <value>One of the enumeration values that represents the conformance mode used by this reader.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -182,7 +182,7 @@
 
 ## Remarks
 
-Useful in scenarios where the semantic value decoder needs to be determined at runtime.
+Useful in scenarios where the semantic value decoder needs to be determined at run time.
 
           ]]></format>
         </remarks>
@@ -298,7 +298,7 @@ The next value uses a CBOR encoding that is not valid under the current conforma
 
 ## Remarks
 
-The method accepts indefinite length strings, which it will concatenate to a single string.
+The method accepts indefinite length strings, which it concatenates to a single string.
 
           ]]></format>
         </remarks>
@@ -569,7 +569,7 @@ The next value uses a CBOR encoding that is not valid under the current conforma
         <Parameter Name="disableConformanceModeChecks" Type="System.Boolean" />
       </Parameters>
       <Docs>
-        <param name="disableConformanceModeChecks">Disable conformance mode validation for the read value, equivalent to using <see cref="F:System.Formats.Cbor.CborConformanceMode.Lax" />.</param>
+        <param name="disableConformanceModeChecks"><see langword="true" /> to disable conformance mode validation for the read value, equivalent to using <see cref="F:System.Formats.Cbor.CborConformanceMode.Lax" />; otherwise, <see langword="false" />.</param>
         <summary>Reads the next CBOR data item, returning a <see cref="T:System.ReadOnlyMemory`1" /> view of the encoded value. For indefinite length encodings this includes the break byte.</summary>
         <returns>A view of the encoded value as a contiguous region of memory.</returns>
         <remarks>To be added.</remarks>
@@ -1044,7 +1044,7 @@ The next value uses a CBOR encoding that is not valid under the current conforma
 ## Remarks
 
 Map contents are consumed as if they were arrays twice the length of the map's declared size.
-For instance, a map of size `1` containing a key of type int with a value of type string must be consumed by successive calls to <xref:System.Formats.Cbor.CborReader.ReadInt32> and <xref:System.Formats.Cbor.CborReader.ReadTextString>.
+For instance, a map of size `1` containing a key of type `int` with a value of type `string` must be consumed by successive calls to <xref:System.Formats.Cbor.CborReader.ReadInt32> and <xref:System.Formats.Cbor.CborReader.ReadTextString>.
 It is up to the caller to keep track of whether the next value is a key or a value.
 Fundamentally, this is a technical restriction stemming from the fact that CBOR allows keys of arbitrary type, for instance a map can contain keys that are maps themselves.
 

xml/System.Formats.Cbor/CborWriter.xml

@@ -737,7 +737,7 @@ The written data is not accepted under the current conformance mode.</exception>
       </Parameters>
       <Docs>
         <param name="value">The value to write</param>
-        <summary>Writes an value as a signed integer encoding (major types 0,1)</summary>
+        <summary>Writes a value as a signed integer encoding (major types 0,1)</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.InvalidOperationException">Writing a new value exceeds the definite length of the parent data item.
 
@@ -1189,7 +1189,7 @@ The written data is not accepted under the current conformance mode.</exception>
       </Parameters>
       <Docs>
         <param name="value">The value to write</param>
-        <summary>Writes anvalue as an unsigned integer encoding (major type 0).</summary>
+        <summary>Writes a value as an unsigned integer encoding (major type 0).</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.InvalidOperationException">Writing a new value exceeds the definite length of the parent data item.
 
@@ -1228,7 +1228,7 @@ The written data is not accepted under the current conformance mode.</exception>
       </Parameters>
       <Docs>
         <param name="value">The value to write</param>
-        <summary>Writes an value as an unsigned integer encoding (major type 0).</summary>
+        <summary>Writes a value as an unsigned integer encoding (major type 0).</summary>
         <remarks>To be added.</remarks>
         <exception cref="T:System.InvalidOperationException">Writing a new value exceeds the definite length of the parent data item.
 

xml/System.Globalization/CompareInfo.xml

@@ -2233,7 +2233,7 @@ Use <xref:System.Globalization.CompareInfo.GetSortKeyLength(System.ReadOnlySpan{
 
 ## Remarks
 
-This method has greater overhead than other <xref:System.Globalization.CompareInfo.IndexOf%2A> overloads which don't take a `matchLength` argument. Call this overload only if you require the match length information.
+This method has greater overhead than other <xref:System.Globalization.CompareInfo.IndexOf%2A> overloads that don't take a `matchLength` argument. Call this overload only if you require the match length information.
 
           ]]></format>
         </remarks>
@@ -3119,7 +3119,7 @@ This method has greater overhead than other <xref:System.Globalization.CompareIn
 
 ## Remarks
 
-This method has greater overhead than other <xref:System.Globalization.CompareInfo.IsPrefix(System.String,System.String,System.Globalization.CompareOptions)> overloads which don't take a `matchLength` argument. Call this overload only if you require the match length information.
+This method has greater overhead than other <xref:System.Globalization.CompareInfo.IsPrefix(System.String,System.String,System.Globalization.CompareOptions)> overloads that don't take a `matchLength` argument. Call this overload only if you require the match length information.
 
           ]]></format>
         </remarks>
@@ -3589,7 +3589,7 @@ This method has greater overhead than other <xref:System.Globalization.CompareIn
 
 ## Remarks
 
-This method has greater overhead than other <xref:System.Globalization.CompareInfo.IsSuffix(System.String,System.String,System.Globalization.CompareOptions)> overloads which don't take a `matchLength` argument. Call this overload only if you require the match length information.
+This method has greater overhead than other <xref:System.Globalization.CompareInfo.IsSuffix(System.String,System.String,System.Globalization.CompareOptions)> overloads that don't take a `matchLength` argument. Call this overload only if you require the match length information.
 
           ]]></format>
         </remarks>
@@ -4261,7 +4261,7 @@ This method has greater overhead than other <xref:System.Globalization.CompareIn
 
 ## Remarks
 
-This method has greater overhead than other <xref:System.Globalization.CompareInfo.LastIndexOf%2A> overloads which don't take a `matchLength` argument. Call this overload only if you require the match length information.
+This method has greater overhead than other <xref:System.Globalization.CompareInfo.LastIndexOf%2A> overloads that don't take a `matchLength` argument. Call this overload only if you require the match length information.
 
           ]]></format>
         </remarks>