Fix malformed XML with orphaned para tags and apply proper formatting guidelines

Copilot · jeffhandley · Copilot · commit 678a55a97d62 · 2025-06-18T01:19:26.000Z
Co-authored-by: jeffhandley &lt;1031940+jeffhandley@users.noreply.github.com&gt;
diff --git a/xml/System.Collections.Concurrent/ConcurrentDictionary`2+AlternateLookup`1.xml b/xml/System.Collections.Concurrent/ConcurrentDictionary`2+AlternateLookup`1.xml
@@ -127,8 +127,9 @@
       <Docs>
         <param name="key">The alternate key of the value to get or set.</param>
         <summary>Gets or sets the value associated with the specified alternate key.</summary>
-        <value>The value associated with the specified alternate key. If the specified alternate key is not found, a get operation throws</para>
-          <para>a <see cref="T:System.Collections.Generic.KeyNotFoundException" />, and a set operation creates a new element with the specified key.</value>
+        <value>
+            The value associated with the specified alternate key. If the specified alternate key is not found, a get operation throws a <see cref="T:System.Collections.Generic.KeyNotFoundException" />, and a set operation creates a new element with the specified key.
+        </value>
         <remarks>To be added.</remarks>
         <exception cref="T:System.ArgumentNullException">
           <paramref name="key" /> is <see langword="null" />.</exception>
@@ -241,9 +242,13 @@
       </Parameters>
       <Docs>
         <param name="key">The alternate key of the value to get.</param>
-        <param name="actualKey">When this method returns, contains the actual key associated with the alternate key, if the key is found;</para>
-          <para>otherwise, the default value for the type of the key parameter.</param>
-        <param name="value">When this method returns, contains the value associated with the specified key, if the key is found; otherwise, the default value for the type of the value parameter.</param>
+        <param name="actualKey">
+            When this method returns, contains the actual key associated with the alternate key, if the key is found;
+            otherwise, the default value for the type of the key parameter.
+        </param>
+        <param name="value">
+            When this method returns, contains the value associated with the specified key, if the key is found; otherwise, the default value for the type of the value parameter.
+        </param>
         <summary>Gets the value associated with the specified alternate key.</summary>
         <returns>
           <see langword="true" /> if an entry was found; otherwise, <see langword="false" />.</returns>
@@ -329,8 +334,10 @@
         <param name="key">The alternate key of the element to remove.</param>
         <param name="actualKey">The removed key.</param>
         <param name="value">The removed element.</param>
-        <summary>Removes the value with the specified alternate key from the <see cref="T:System.Collections.Generic.Dictionary`2" />,</para>
-          <para>and copies the associated key and element to the value parameter.</summary>
+        <summary>
+            Removes the value with the specified alternate key from the <see cref="T:System.Collections.Generic.Dictionary`2" />,
+            and copies the associated key and element to the value parameter.
+        </summary>
         <returns>
           <see langword="true" /> if the element is successfully found and removed; otherwise, <see langword="false" />.</returns>
         <remarks>To be added.</remarks>
diff --git a/xml/System.Collections.Concurrent/ConcurrentDictionary`2.xml b/xml/System.Collections.Concurrent/ConcurrentDictionary`2.xml
@@ -1014,9 +1014,10 @@
           Gets an instance of a type that may be used to perform operations on a <see cref="T:System.Collections.Concurrent.ConcurrentDictionary`2" /> using a <typeparamref name="TAlternateKey" /> as a key instead of a <typeparamref name="TKey" />.
         </summary>
         <returns>The created lookup instance.</returns>
-        <remarks>This instance must be using a comparer that implements <see cref="T:System.Collections.Generic.IAlternateEqualityComparer`2" /> with</para>
-          <para>
-            <typeparamref name="TAlternateKey" /> and <typeparamref name="TKey" />. If it doesn't, an exception will be thrown.</remarks>
+        <remarks>
+            This instance must be using a comparer that implements <see cref="T:System.Collections.Generic.IAlternateEqualityComparer`2" /> with
+            <typeparamref name="TAlternateKey" /> and <typeparamref name="TKey" />. If it doesn't, an exception will be thrown.
+        </remarks>
         <exception cref="T:System.InvalidOperationException">This instance's comparer is not compatible with <typeparamref name="TAlternateKey" />.</exception>
       </Docs>
     </Member>
diff --git a/xml/System.Collections.Frozen/FrozenDictionary`2+AlternateLookup`1.xml b/xml/System.Collections.Frozen/FrozenDictionary`2+AlternateLookup`1.xml
@@ -123,8 +123,9 @@
       <Docs>
         <param name="key">The alternate key of the value to get or set.</param>
         <summary>Gets or sets the value associated with the specified alternate key.</summary>
-        <value>The value associated with the specified alternate key. If the specified alternate key is not found, a get operation throws</para>
-          <para>a <see cref="T:System.Collections.Generic.KeyNotFoundException" />, and a set operation creates a new element with the specified key.</value>
+        <value>
+            The value associated with the specified alternate key. If the specified alternate key is not found, a get operation throws a <see cref="T:System.Collections.Generic.KeyNotFoundException" />, and a set operation creates a new element with the specified key.
+        </value>
         <remarks>To be added.</remarks>
         <exception cref="T:System.ArgumentNullException">
           <paramref name="key" /> is <see langword="null" />.</exception>
@@ -160,8 +161,10 @@
       </Parameters>
       <Docs>
         <param name="key">The alternate key of the value to get.</param>
-        <param name="value">When this method returns, contains the value associated with the specified key, if the key is found;</para>
-          <para>otherwise, the default value for the type of the value parameter.</param>
+        <param name="value">
+            When this method returns, contains the value associated with the specified key, if the key is found;
+            otherwise, the default value for the type of the value parameter.
+        </param>
         <summary>Gets the value associated with the specified alternate key.</summary>
         <returns>
           <see langword="true" /> if an entry was found; otherwise, <see langword="false" />.</returns>
diff --git a/xml/System.Text.Json.Serialization.Metadata/JsonParameterInfo.xml b/xml/System.Text.Json.Serialization.Metadata/JsonParameterInfo.xml
@@ -50,8 +50,10 @@
       <Docs>
         <summary>Gets a custom attribute provider for the current parameter.</summary>
         <value>To be added.</value>
-        <remarks>When resolving metadata via the built-in resolvers this will be populated with</para>
-          <para>the underlying <see cref="T:System.Reflection.ParameterInfo" /> of the constructor metadata.</remarks>
+        <remarks>
+            When resolving metadata via the built-in resolvers this will be populated with
+            the underlying <see cref="T:System.Reflection.ParameterInfo" /> of the constructor metadata.
+        </remarks>
       </Docs>
     </Member>
     <Member MemberName="DeclaringType">
@@ -145,8 +147,10 @@
       <Docs>
         <summary>Gets a value indicating whether the parameter represents a required or init-only member initializer.</summary>
         <value>To be added.</value>
-        <remarks>Only returns <see langword="true" /> for source-generated metadata that can only access</para>
-          <para>required or init-only member initializers using object initialize expressions.</remarks>
+        <remarks>
+            Only returns <see langword="true" /> for source-generated metadata that can only access
+            required or init-only member initializers using object initialize expressions.
+        </remarks>
       </Docs>
     </Member>
     <Member MemberName="IsNullable">
diff --git a/xml/System.Text.Json/JsonSerializer.xml b/xml/System.Text.Json/JsonSerializer.xml
@@ -381,14 +381,13 @@ There is remaining data in the string beyond a single JSON value.</exception>
         <param name="jsonTypeInfo">Metadata about the type to convert.</param>
         <summary>Reads one JSON value (including objects or arrays) from the provided reader into an instance specified by the <paramref name="jsonTypeInfo" />.</summary>
         <returns>A <paramref name="jsonTypeInfo" /> representation of the JSON value.</returns>
-        <remarks>If the <see cref="P:System.Text.Json.Utf8JsonReader.TokenType" /> property of <paramref name="reader" /> is <see cref="F:System.Text.Json.JsonTokenType.PropertyName" /> or <see cref="F:System.Text.Json.JsonTokenType.None" />, the reader will be advanced by one call to <see cref="M:System.Text.Json.Utf8JsonReader.Read" /> to determine the start of the value.
- </para>
-          <para> Upon completion of this method, <paramref name="reader" /> will be positioned at the final token in the JSON value. If an exception is thrown, the reader is reset to the state it was in when the method was called.
- </para>
-          <para> This method makes a copy of the data the reader acted on, so there is no caller requirement to maintain data integrity beyond the return of this method.
- </para>
-          <para> The <see cref="T:System.Text.Json.JsonReaderOptions" /> used to create the instance of the <see cref="T:System.Text.Json.Utf8JsonReader" /> take precedence over the <see cref="T:System.Text.Json.JsonSerializerOptions" /> when they conflict.
- Hence, <see cref="P:System.Text.Json.JsonReaderOptions.AllowTrailingCommas" />, <see cref="P:System.Text.Json.JsonReaderOptions.MaxDepth" />, and <see cref="P:System.Text.Json.JsonReaderOptions.CommentHandling" /> are used while reading.</remarks>
+        <remarks>
+            If the <see cref="P:System.Text.Json.Utf8JsonReader.TokenType" /> property of <paramref name="reader" /> is <see cref="F:System.Text.Json.JsonTokenType.PropertyName" /> or <see cref="F:System.Text.Json.JsonTokenType.None" />, the reader will be advanced by one call to <see cref="M:System.Text.Json.Utf8JsonReader.Read" /> to determine the start of the value.
+            Upon completion of this method, <paramref name="reader" /> will be positioned at the final token in the JSON value. If an exception is thrown, the reader is reset to the state it was in when the method was called.
+            This method makes a copy of the data the reader acted on, so there is no caller requirement to maintain data integrity beyond the return of this method.
+            The <see cref="T:System.Text.Json.JsonReaderOptions" /> used to create the instance of the <see cref="T:System.Text.Json.Utf8JsonReader" /> take precedence over the <see cref="T:System.Text.Json.JsonSerializerOptions" /> when they conflict.
+            Hence, <see cref="P:System.Text.Json.JsonReaderOptions.AllowTrailingCommas" />, <see cref="P:System.Text.Json.JsonReaderOptions.MaxDepth" />, and <see cref="P:System.Text.Json.JsonReaderOptions.CommentHandling" /> are used while reading.
+        </remarks>
         <exception cref="T:System.Text.Json.JsonException">The JSON is invalid,
              <paramref name="jsonTypeInfo" /> is not compatible with the JSON,
              or a value could not be read from the reader.</exception>
