Apply suggestions from code review

Co-authored-by: Genevieve Warren <[email protected]>

Author: tannergooding

xml/System/Environment+ProcessCpuUsage.xml

@@ -71,7 +71,7 @@
         <ReturnType>System.TimeSpan</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Gets the amount of time the process has spent utilizing the CPU including the process time spent in the application code and the process time spent in the operating system code.</summary>
+        <summary>Gets the amount of time the process has spent utilizing the CPU, including the process time spent in the application code and in the operating system code.</summary>
         <value>To be added.</value>
         <remarks>To be added.</remarks>
       </Docs>

xml/System/Guid.xml

@@ -604,8 +604,8 @@
       </ReturnValue>
       <Docs>
         <summary>Gets a <see cref="T:System.Guid" /> where all bits are set.</summary>
-        <value>To be added.</value>
-        <remarks>This returns the value: FFFFFFFF-FFFF-FFFF-FFFF-FFFFFFFFFFFF</remarks>
+        <value>A GUID with a value of FFFFFFFF-FFFF-FFFF-FFFF-FFFFFFFFFFFF.</value>
+        <remarks>To be added.</remarks>
       </Docs>
     </Member>
     <MemberGroup MemberName="CompareTo">
@@ -849,8 +849,8 @@
         <summary>Creates a new <see cref="T:System.Guid" /> according to RFC 9562, following the Version 7 format.</summary>
         <returns>A new <see cref="T:System.Guid" /> according to RFC 9562, following the Version 7 format.</returns>
         <remarks>
-          <para>This uses <see cref="P:System.DateTimeOffset.UtcNow" /> to determine the Unix Epoch timestamp source.</para>
-          <para>This seeds the rand_a and rand_b sub-fields with random data.</para>
+          <para>This method uses <see cref="P:System.DateTimeOffset.UtcNow" /> to determine the Unix Epoch timestamp source.</para>
+          <para>This method seeds the rand_a and rand_b sub-fields with random data.</para>
         </remarks>
       </Docs>
     </Member>
@@ -879,11 +879,11 @@
         <Parameter Name="timestamp" Type="System.DateTimeOffset" Index="0" FrameworkAlternate="net-9.0" />
       </Parameters>
       <Docs>
-        <param name="timestamp">The date time offset used to determine the Unix Epoch timestamp.</param>
+        <param name="timestamp">The date-time offset used to determine the Unix Epoch timestamp.</param>
         <summary>Creates a new <see cref="T:System.Guid" /> according to RFC 9562, following the Version 7 format.</summary>
         <returns>A new <see cref="T:System.Guid" /> according to RFC 9562, following the Version 7 format.</returns>
         <remarks>
-          <para>This seeds the rand_a and rand_b sub-fields with random data.</para>
+          <para>This method seeds the rand_a and rand_b sub-fields with random data.</para>
         </remarks>
         <exception cref="T:System.ArgumentOutOfRangeException">
           <paramref name="timestamp" /> represents an offset prior to <see cref="F:System.DateTimeOffset.UnixEpoch" />.</exception>
@@ -3021,8 +3021,8 @@ Because the `provider` parameter is ignored, you cannot use it to provide a cust
         <summary>Gets the value of the variant field for the <see cref="T:System.Guid" />.</summary>
         <value>To be added.</value>
         <remarks>
-          <para>This corresponds to the most significant 4 bits of the 8th byte: 00000000-0000-0000-F000-000000000000. The "don't-care" bits are not masked out.</para>
-          <para>See RFC 9562 for more information on how to interpret this value.</para>
+          <para>This property corresponds to the most significant 4 bits of the 8th byte: 00000000-0000-0000-F000-000000000000. The "don't-care" bits are not masked out.</para>
+          <para>For more information on how to interpret this value, see <see href="https://www.rfc-editor.org/rfc/rfc9562.html">RFC 9562</see>.</para>
         </remarks>
       </Docs>
     </Member>
@@ -3051,8 +3051,8 @@ Because the `provider` parameter is ignored, you cannot use it to provide a cust
         <summary>Gets the value of the version field for the <see cref="T:System.Guid" />.</summary>
         <value>To be added.</value>
         <remarks>
-          <para>This corresponds to the most significant 4 bits of the 6th byte: 00000000-0000-F000-0000-000000000000.</para>
-          <para>See RFC 9562 for more information on how to interpret this value.</para>
+          <para>This property corresponds to the most significant 4 bits of the 6th byte: 00000000-0000-F000-0000-000000000000.</para>
+          <para>For more information on how to interpret this value, see <see href="https://www.rfc-editor.org/rfc/rfc9562.html">RFC 9562</see>.</para>
         </remarks>
       </Docs>
     </Member>

xml/System/MemoryExtensions+SpanSplitEnumerator`1.xml

@@ -62,7 +62,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets the current element of the enumeration.</summary>
-        <value>Returns a <see cref="T:System.Range" /> instance that indicates the bounds of the current element withing the source span.</value>
+        <value>A <see cref="T:System.Range" /> instance that indicates the bounds of the current element withing the source span.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -90,7 +90,7 @@
       <Parameters />
       <Docs>
         <summary>Gets an enumerator that allows for iteration over the split span.</summary>
-        <returns>Returns a <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" /> that can be used to iterate over the split span.</returns>
+        <returns>A <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" /> that can be used to iterate over the split span.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System/MemoryExtensions.xml

@@ -9339,7 +9339,7 @@ Invalid sequences will be represented in the enumeration by <xref:System.Text.Ru
         <summary>
           <para>Returns a type that allows for enumeration of each element within a split span using the provided separator span.</para>
         </summary>
-        <returns>Returns a <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" />.</returns>
+        <returns>An enumerator that can be used to enumerate each element in a split span using the provided separator span.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -9404,7 +9404,7 @@ Invalid sequences will be represented in the enumeration by <xref:System.Text.Ru
         <summary>
           <para>Returns a type that allows for enumeration of each element within a split span using the provided separator character.</para>
         </summary>
-        <returns>Returns a <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" />.</returns>
+        <returns>An enumerator that can be used to enumerate each element in a split span using the provided separator character.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -9558,14 +9558,14 @@ Invalid sequences will be represented in the enumeration by <xref:System.Text.Ru
       <Docs>
         <typeparam name="T">The type of the elements.</typeparam>
         <param name="source">The source span to be enumerated.</param>
-        <param name="separators">The <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" /> to be used to split the provided span.</param>
+        <param name="separators">The <see cref="T:System.Buffers.SearchValues`1" /> to be used to split the provided span.</param>
         <summary>
-          <para>Returns a type that allows for enumeration of each element within a split span using the provided <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" />.</para>
+          <para>Returns a type that allows for enumeration of each element within a split span using the provided separator characters.</para>
         </summary>
-        <returns>Returns a <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" />.</returns>
+        <returns>An enumerator that can be used to enumerate each element in a split span using the provided separator characters.</returns>
         <remarks>
-          <para>Unlike <see cref="M:System.MemoryExtensions.SplitAny``1(System.ReadOnlySpan{``0},System.ReadOnlySpan{``0})" />, the <paramref name="separators" /> is not checked for being empty.</para>
-          <para>An empty <paramref name="separators" /> will result in no separators being found, regardless of the type of <typeparamref name="T" />, whereas <see cref="M:System.MemoryExtensions.SplitAny``1(System.ReadOnlySpan{``0},System.ReadOnlySpan{``0})" /> will use all Unicode whitespace characters as separators if <paramref name="separators" /> is empty and <typeparamref name="T" /> is <see cref="T:System.Char" />.</para>
+          <para>Unlike <see cref="M:System.MemoryExtensions.SplitAny``1(System.ReadOnlySpan{``0},System.ReadOnlySpan{``0})" />, <paramref name="separators" /> is not checked for being empty.</para>
+          <para>An empty <paramref name="separators" /> argument will result in no separators being found, regardless of the type of <typeparamref name="T" />, whereas <see cref="M:System.MemoryExtensions.SplitAny``1(System.ReadOnlySpan{``0},System.ReadOnlySpan{``0})" /> uses all Unicode whitespace characters as separators if <paramref name="separators" /> is empty and <typeparamref name="T" /> is <see cref="T:System.Char" />.</para>
         </remarks>
       </Docs>
     </Member>
@@ -9633,7 +9633,7 @@ Invalid sequences will be represented in the enumeration by <xref:System.Text.Ru
         <summary>
           <para>Returns a type that allows for enumeration of each element within a split span using any of the provided elements.</para>
         </summary>
-        <returns>Returns a <see cref="T:System.MemoryExtensions.SpanSplitEnumerator`1" />.</returns>
+        <returns>An enumerator that can be used to enumerate each element in a split span using the provided separator characters.</returns>
         <remarks>
           <para>If <typeparamref name="T" /> is <see cref="T:System.Char" /> and if <paramref name="separators" /> is empty, all Unicode whitespace characters are used as the separators. This matches the behavior of when <see cref="M:System.String.Split(System.Char[])" /> and related overloads are used with an empty separator array, or when <see cref="M:System.MemoryExtensions.SplitAny(System.ReadOnlySpan{System.Char},System.Span{System.Range},System.ReadOnlySpan{System.Char},System.StringSplitOptions)" /> is used with an empty separator span.</para>
         </remarks>