Intellisense review fixes - Part 1 (dotnet#3482)

* Intellisense review fixes

* fix bad xml

Author: mairaw

xml/Microsoft.Extensions.DependencyInjection/HttpClientBuilderExtensions.xml

@@ -48,7 +48,7 @@
         <returns>An <see cref="T:Microsoft.Extensions.DependencyInjection.IHttpClientBuilder" /> that can be used to configure the client.</returns>
         <remarks>
           <para>
-            The <see paramref="configureHandler" /> delegate should return a new instance of the message handler each time it
+            The <paramref name="configureHandler" /> delegate should return a new instance of the message handler each time it
             is invoked.
             </para>
           <para>
@@ -84,7 +84,7 @@
         <param name="configureHandler">A delegate that is used to create a <see cref="T:System.Net.Http.DelegatingHandler" />.</param>
         <summary>Adds a delegate that will be used to create an additional message handler for a named <see cref="T:System.Net.Http.HttpClient" />.</summary>
         <returns>An <see cref="T:Microsoft.Extensions.DependencyInjection.IHttpClientBuilder" /> that can be used to configure the client.</returns>
-        <remarks>The <see paramref="configureHandler" /> delegate should return a new instance of the message handler each time it
+        <remarks>The <paramref name="configureHandler" /> delegate should return a new instance of the message handler each time it
             is invoked.</remarks>
       </Docs>
     </Member>
@@ -470,7 +470,7 @@
         <returns>An <see cref="T:Microsoft.Extensions.DependencyInjection.IHttpClientBuilder" /> that can be used to configure the client.</returns>
         <remarks>
           <para>
-            The <see paramref="configureHandler" /> delegate should return a new instance of the message handler each time it
+            The <paramref name="configureHandler" /> delegate should return a new instance of the message handler each time it
             is invoked.
             </para>
           <para>
@@ -507,7 +507,7 @@
         <summary>Adds a delegate that will be used to configure the primary <see cref="T:System.Net.Http.HttpMessageHandler" /> for a
             named <see cref="T:System.Net.Http.HttpClient" />.</summary>
         <returns>An <see cref="T:Microsoft.Extensions.DependencyInjection.IHttpClientBuilder" /> that can be used to configure the client.</returns>
-        <remarks>The <see paramref="configureHandler" /> delegate should return a new instance of the message handler each time it
+        <remarks>The <paramref name="configureHandler" /> delegate should return a new instance of the message handler each time it
             is invoked.</remarks>
       </Docs>
     </Member>

xml/System.Buffers/SequenceReader`1.xml

@@ -641,7 +641,7 @@
       <Docs>
         <param name="delimiters">The delimiters to search for.</param>
         <param name="advancePastDelimiter">
-          <see langword="true" /> to move past the first instance of any of the given <paramref name="delimiters" />; <see lanword="false" /> to not move past the delimiter.</param>
+          <see langword="true" /> to move past the first instance of any of the given <paramref name="delimiters" />; <see langword="false" /> to not move past the delimiter.</param>
         <summary>Searches for any of a number of specified delimiters and optionally advances past the first one to be found.</summary>
         <returns>
           <see langword="true" /> if any of the given <paramref name="delimiters" /> was found; otherwise, <see langword="false" />.</returns>

xml/System.Collections/CollectionBase.xml

@@ -329,9 +329,7 @@ A <xref:System.Collections.CollectionBase> instance is always modifiable. See <x
       </ReturnValue>
       <Docs>
         <summary>Gets the number of elements contained in the <see cref="T:System.Collections.CollectionBase" /> instance. This property cannot be overridden.</summary>
-        <value>The number of elements contained in the <see cref="T:System.Collections.CollectionBase" /> instance.  
-  
- Retrieving the value of this property is an `O(1)` operation.</value>
+        <value>The number of elements contained in the <see cref="T:System.Collections.CollectionBase" /> instance. Retrieving the value of this property is an O(1) operation.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   
@@ -341,7 +339,7 @@ A <xref:System.Collections.CollectionBase> instance is always modifiable. See <x
  [!code-cpp[System.Collections.CollectionBase#1](~/samples/snippets/cpp/VS_Snippets_CLR_System/system.Collections.CollectionBase/CPP/collectionbase.cpp#1)]
  [!code-csharp[System.Collections.CollectionBase#1](~/samples/snippets/csharp/VS_Snippets_CLR_System/system.Collections.CollectionBase/CS/collectionbase.cs#1)]
  [!code-vb[System.Collections.CollectionBase#1](~/samples/snippets/visualbasic/VS_Snippets_CLR_System/system.Collections.CollectionBase/VB/collectionbase.vb#1)]  
-  
+   
  ]]></format>
         </remarks>
       </Docs>
@@ -453,9 +451,7 @@ A <xref:System.Collections.CollectionBase> instance is always modifiable. See <x
       </ReturnValue>
       <Docs>
         <summary>Gets an <see cref="T:System.Collections.ArrayList" /> containing the list of elements in the <see cref="T:System.Collections.CollectionBase" /> instance.</summary>
-        <value>An <see cref="T:System.Collections.ArrayList" /> representing the <see cref="T:System.Collections.CollectionBase" /> instance itself.  
-  
- Retrieving the value of this property is an `O(1)` operation.</value>
+        <value>An <see cref="T:System.Collections.ArrayList" /> representing the <see cref="T:System.Collections.CollectionBase" /> instance itself. Retrieving the value of this property is an O(1) operation.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   

xml/System.Collections/ReadOnlyCollectionBase.xml

@@ -157,9 +157,7 @@ A <xref:System.Collections.ReadOnlyCollectionBase> instance is always read-only.
       </ReturnValue>
       <Docs>
         <summary>Gets the number of elements contained in the <see cref="T:System.Collections.ReadOnlyCollectionBase" /> instance.</summary>
-        <value>The number of elements contained in the <see cref="T:System.Collections.ReadOnlyCollectionBase" /> instance.  
-  
- Retrieving the value of this property is an `O(1)` operation.</value>
+        <value>The number of elements contained in the <see cref="T:System.Collections.ReadOnlyCollectionBase" /> instance. Retrieving the value of this property is an O(1) operation.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   

xml/System.Data.Common/DbCommand.xml

@@ -645,7 +645,7 @@
       <Parameters />
       <Docs>
         <summary>Asynchronously diposes the command object.</summary>
-        <returns>A <see cref="P:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
+        <returns>A <see cref="T:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   
@@ -1495,7 +1495,7 @@
         <Parameter Name="cancellationToken" Type="System.Threading.CancellationToken" Index="0" FrameworkAlternate="netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="System.Threading.CancellationToken.None" />.</param>
+        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="P:System.Threading.CancellationToken.None" />.</param>
         <summary>Asynchronously creates a prepared (or compiled) version of the command on the data source.</summary>
         <returns>A <see cref="P:System.Threading.Tasks.Task" /> representing the asynchronous operation.</returns>
         <remarks>

xml/System.Data.Common/DbConnection.xml

@@ -294,7 +294,7 @@
         <Parameter Name="cancellationToken" Type="System.Threading.CancellationToken" Index="0" FrameworkAlternate="netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="System.Threading.CancellationToken.None" />.</param>
+        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="P:System.Threading.CancellationToken.None" />.</param>
         <summary>Asynchronously begins a database transaction.</summary>
         <returns>A task whose <see cref="P:System.Threading.Tasks.Task`1.Result" /> property is an object representing the new transaction.</returns>
         <remarks>
@@ -337,7 +337,7 @@
       </Parameters>
       <Docs>
         <param name="isolationLevel">One of the enumeration values that specifies the isolation level for the transaction to use.</param>
-        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="System.Threading.CancellationToken.None" />.</param>
+        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="P:System.Threading.CancellationToken.None" />.</param>
         <summary>Asynchronously begins a database transaction.</summary>
         <returns>A task whose <see cref="P:System.Threading.Tasks.Task`1.Result" /> property is an object representing the new transaction.</returns>
         <remarks>
@@ -427,7 +427,7 @@
       </Parameters>
       <Docs>
         <param name="databaseName">The name of the database for the connection to use.</param>
-        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="System.Threading.CancellationToken.None" />.</param>
+        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="P:System.Threading.CancellationToken.None" />.</param>
         <summary>Asynchronously changes the current database for an open connection.</summary>
         <returns>A task representing the asynchronous operation.</returns>
         <remarks>
@@ -943,7 +943,7 @@
       <Parameters />
       <Docs>
         <summary>Asynchronously diposes the connection object.</summary>
-        <returns>A <see cref="P:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
+        <returns>A <see cref="T:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   

xml/System.Data.Common/DbDataReader.xml

@@ -348,7 +348,7 @@
       <Parameters />
       <Docs>
         <summary>Asynchronously releases all resources used by the current instance of the <see cref="T:System.Data.Common.DbDataReader" /> class.</summary>
-        <returns>A <see cref="P:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
+        <returns>A <see cref="T:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   

xml/System.Data.Common/DbTransaction.xml

@@ -138,7 +138,7 @@
         <Parameter Name="cancellationToken" Type="System.Threading.CancellationToken" Index="0" FrameworkAlternate="netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="System.Threading.CancellationToken.None" />.</param>
+        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="P:System.Threading.CancellationToken.None" />.</param>
         <summary>Asynchronously commits the database transaction.</summary>
         <returns>A <see cref="P:System.Threading.Tasks.Task" /> representing the asynchronous operation.</returns>
         <remarks>
@@ -382,7 +382,7 @@
       <Parameters />
       <Docs>
         <summary>Asynchronously diposes the transaction object.</summary>
-        <returns>A <see cref="P:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
+        <returns>A <see cref="T:System.Threading.Tasks.ValueTask" /> representing the asynchronous operation.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   
@@ -499,7 +499,7 @@
         <Parameter Name="cancellationToken" Type="System.Threading.CancellationToken" Index="0" FrameworkAlternate="netcore-3.0;netstandard-2.1" />
       </Parameters>
       <Docs>
-        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="System.Threading.CancellationToken.None" />.</param>
+        <param name="cancellationToken">An optional token to cancel the asynchronous operation. The default value is <see cref="P:System.Threading.CancellationToken.None" />.</param>
         <summary>Asynchronously rolls back a transaction from a pending state.</summary>
         <returns>A task representing the asynchronous operation.</returns>
         <remarks>

xml/System.DirectoryServices.ActiveDirectory/ActiveDirectorySchedule.xml

@@ -158,19 +158,26 @@
  </term><description> The source is available for replication from 45 to 59 minutes after the hour.  
 
  </description></item></list>  
+</value>
+        <remarks>
+          <format type="text/markdown"><![CDATA[  
+  
+## Examples  
 
- The following C# example shows how to use this property to determine if the source is available for replication at 15:50 Coordinated Universal Time on Tuesday.  
+The following example shows how to use this property to determine if the source is available for replication at 15:50 Coordinated Universal Time on Tuesday:
   
 ```csharp  
-Boolean isAvailable = scheduleObject.RawSchedule[2, 15, 3];  
+bool isAvailable = scheduleObject.RawSchedule[2, 15, 3];
 ```  
   
- The following C# example shows how to use this property to calculate the 15-minute interval at runtime by dividing the minutes by 15.  
+The following example shows how to use this property to calculate the 15-minute interval at runtime by dividing the minutes by 15: 
   
 ```csharp  
-Boolean isAvailable = scheduleObject.RawSchedule[2, 15, (Int32)50/15];  
-```</value>
-        <remarks>To be added.</remarks>
+bool isAvailable = scheduleObject.RawSchedule[2, 15, (Int32)50/15];  
+```
+
+ ]]></format>
+        </remarks>
       </Docs>
     </Member>
     <Member MemberName="ResetSchedule">

xml/System.DirectoryServices.ActiveDirectory/ActiveDirectorySchemaClass.xml

@@ -396,10 +396,11 @@
         <ReturnType>System.String</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Gets the ldapDisplayName of the <see cref="T:System.DirectoryServices.ActiveDirectory.ActiveDirectorySchemaClass" /> object. For more information, see the topic [LDAP-Display-Name](https://msdn.microsoft.com/library/default.asp?url=/library/adschema/adschema/a_ldapdisplayname.asp) in the MSDN Library at [https://msdn.microsoft.com](https://msdn.microsoft.com/).</summary>
-        <value>A <see cref="T:System.String" /> that contains the ldapDisplayName of the <see cref="T:System.DirectoryServices.ActiveDirectory.ActiveDirectorySchemaClass" /> object. For more information, see the topic [LDAP-Display-Name](https://msdn.microsoft.com/library/default.asp?url=/library/adschema/adschema/a_ldapdisplayname.asp) in the MSDN Library at [https://msdn.microsoft.com](https://msdn.microsoft.com/).</value>
+        <summary>Gets the ldapDisplayName of the <see cref="T:System.DirectoryServices.ActiveDirectory.ActiveDirectorySchemaClass" /> object.</summary>
+        <value>A string that contains the ldapDisplayName of the <see cref="T:System.DirectoryServices.ActiveDirectory.ActiveDirectorySchemaClass" /> object.</value>
         <remarks>To be added.</remarks>
         <altmember cref="M:System.DirectoryServices.ActiveDirectory.ActiveDirectorySchemaClass.Save" />
+        <related type="Article" href="/windows/win32/adschema/a-ldapdisplayname">LDAP-Display-Name attribute</related>
       </Docs>
     </Member>
     <Member MemberName="Oid">
@@ -562,7 +563,7 @@
 ## Remarks  
  The <xref:System.DirectoryServices.ActiveDirectory.ActiveDirectorySchemaClass.SchemaGuid%2A> is used in many Active Directory Domain Services configuration operations, such as managing extended rights.  
 
- For more information, see [schemaIDGUID](https://msdn.microsoft.com/library/ee380169.aspx) or [Schema-ID-GUID](/windows/desktop/adschema/a-schemaidguid).
+ For more information, see [schemaIDGUID](https://docs.microsoft.com/openspecs/windows_protocols/ms-tail/5038cdbe-b665-429c-a36e-36a45f576b81) or [Schema-ID-GUID](/windows/desktop/adschema/a-schemaidguid).
   
  ]]></format>
         </remarks>