PR Feedback

Author: sdmaclea

xml/System.Runtime.Loader/AssemblyLoadContext+ContextualReflectionScope.xml

@@ -18,7 +18,7 @@
     </Interface>
   </Interfaces>
   <Docs>
-    <summary>Return type used exclusively for <see cref="M:System.Runtime.Loader.AssemblyLoadContext.EnterContextualReflection" />. Intended to be used as an <see cref="T:System.IDisposable" /> in a <see langword="using" /> block.</summary>
+    <summary>Provides a return type used exclusively for <see cref="M:System.Runtime.Loader.AssemblyLoadContext.EnterContextualReflection" />. It is intended to be used as an <see cref="T:System.IDisposable" /> in a <see langword="using" /> block.</summary>
     <remarks>To be added.</remarks>
   </Docs>
   <Members>

xml/System.Runtime.Loader/AssemblyLoadContext.xml

@@ -124,8 +124,8 @@ This constructor is protected. It can only be called from the <xref:System.Runti
       </Parameters>
       <Docs>
         <param name="isCollectible">
-          <see langword="true" /> to enable garbage collection; otherwise, <see langword="false" />.</param>
-        <summary>Initializes a new instance of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> class with a value that indicates whether garbage collection is enabled. </summary>
+          <see langword="true" /> to enable <see cref="M:System.Runtime.Loader.AssemblyLoadContext.Unload" />; otherwise, <see langword="false" />. The default value is <see langword="false" /> because there is a performance cost associated with enabling unloading.</param>
+        <summary>Initializes a new instance of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> class with a value that indicates whether unloading is enabled. </summary>
         <remarks>
           <format type="text/markdown"><![CDATA[
 
@@ -153,10 +153,10 @@ This constructor is protected. It can only be called from the <xref:System.Runti
         <Parameter Name="isCollectible" Type="System.Boolean" Index="1" FrameworkAlternate="netcore-3.0" />
       </Parameters>
       <Docs>
-        <param name="name">Value for <see cref="P:System.Runtime.Loader.AssemblyLoadContext.Name" /> in the new instance. Value can be <see langword="null" />.</param>
+        <param name="name">The value for <see cref="P:System.Runtime.Loader.AssemblyLoadContext.Name" /> in the new instance. Its value can be <see langword="null" />.</param>
         <param name="isCollectible">
-          <see langword="true" /> to enable garbage collection; otherwise, <see langword="false" />.</param>
-        <summary>Initializes a new instance of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> class with a name and a value that indicates whether garbage collection is enabled.</summary>
+          <see langword="true" /> to enable <see cref="M:System.Runtime.Loader.AssemblyLoadContext.Unload" />; otherwise, <see langword="false" />. The default value is <see langword="false" /> because there is a performance cost associated with enabling unloading.</param>
+        <summary>Initializes a new instance of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> class with a name and a value that indicates whether unloading is enabled.</summary>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -176,8 +176,8 @@ This constructor is protected. It can only be called from the <xref:System.Runti
         <ReturnType>System.Collections.Generic.IEnumerable&lt;System.Runtime.Loader.AssemblyLoadContext&gt;</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Returns an IEnumerable for all <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> instances.</summary>
-        <value>An IEnumerable for all <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> instances.</value>
+        <summary>Returns an <see cref="T:System.Collections.Generic.IEnumerable`1" /> for all <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> instances.</summary>
+        <value>An <see cref="T:System.Collections.Generic.IEnumerable`1" /> for all <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> instances.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -197,8 +197,8 @@ This constructor is protected. It can only be called from the <xref:System.Runti
         <ReturnType>System.Collections.Generic.IEnumerable&lt;System.Reflection.Assembly&gt;</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Returns an IEnumerable for the <see cref="T:System.Reflection.Assembly" /> instances loaded in the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</summary>
-        <value>An IEnumerable for the <see cref="T:System.Reflection.Assembly" /> instances loaded in the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</value>
+        <summary>Returns an <see cref="T:System.Collections.Generic.IEnumerable`1" /> for the <see cref="T:System.Reflection.Assembly" /> instances loaded in the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</summary>
+        <value>An <see cref="T:System.Collections.Generic.IEnumerable`1" /> for the <see cref="T:System.Reflection.Assembly" /> instances loaded in the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -218,8 +218,8 @@ This constructor is protected. It can only be called from the <xref:System.Runti
         <ReturnType>System.Runtime.Loader.AssemblyLoadContext</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>The <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> set by most recent call to <see cref="M:System.Runtime.Loader.AssemblyLoadContext.EnterContextualReflection" />.</summary>
-        <value>The <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> set by most recent call to <see cref="M:System.Runtime.Loader.AssemblyLoadContext.EnterContextualReflection" />. Value can be <see langword="null" />. The value is stored in an  <see cref="T:System.Threading.AsyncLocal`1" />, so the value is unique per asynchronous context.</value>
+        <summary>Gets the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> set by the most recent call to <see cref="M:System.Runtime.Loader.AssemblyLoadContext.EnterContextualReflection" />.</summary>
+        <value>The <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> set by the most recent call to <see cref="M:System.Runtime.Loader.AssemblyLoadContext.EnterContextualReflection" />. This value can be <see langword="null" />. The value is stored in an <see cref="T:System.Threading.AsyncLocal`1" />, so the value is unique per asynchronous context.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[
 
@@ -276,7 +276,7 @@ For more information, see [AssemblyLoadContext.CurrentContextualReflectionContex
       <Parameters />
       <Docs>
         <summary>Sets <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" /> to <see langword="this" /> <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</summary>
-        <returns>A <see cref="T:System.Runtime.Loader.AssemblyLoadContext.ContextualReflectionScope" /> to restore the previous value of <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" />. Intended to be used as an <see cref="T:System.IDisposable" /> in a <see langword="using" /> block.</returns>
+        <returns>An object to restore the previous value of <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" />. It is intended to be used as an <see cref="T:System.IDisposable" /> in a <see langword="using" /> block.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[
 
@@ -306,9 +306,9 @@ For more information, see [AssemblyLoadContext.CurrentContextualReflectionContex
         <Parameter Name="activating" Type="System.Reflection.Assembly" Index="0" FrameworkAlternate="netcore-3.0" />
       </Parameters>
       <Docs>
-        <param name="activating">Assembly which will be used to determine the requested <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> instance.</param>
-        <summary>Sets <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" /> to the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> which loaded activating.</summary>
-        <returns>A <see cref="T:System.Runtime.Loader.AssemblyLoadContext.ContextualReflectionScope" /> to restore the previous value of <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" />. Intended to be used as an <see cref="T:System.IDisposable" /> in a <see langword="using" /> block.</returns>
+        <param name="activating">The assembly that is used to determine the requested <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> instance.</param>
+        <summary>Sets <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" /> to the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" /> which loaded the assembly.</summary>
+        <returns>An object to restore the previous value of <see cref="P:System.Runtime.Loader.AssemblyLoadContext.CurrentContextualReflectionContext" />. It is intended to be used as an <see cref="T:System.IDisposable" /> in a <see langword="using" /> block.</returns>
         <remarks>
           <format type="text/markdown"><![CDATA[
 
@@ -769,8 +769,8 @@ For more information, see [How to use and debug assembly unloadability in .NET C
         <ReturnType>System.String</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Optional name of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</summary>
-        <value>Optional name of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />. Value can be <see langword="null" />.</value>
+        <summary>Get the name of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />.</summary>
+        <value>The name of the <see cref="T:System.Runtime.Loader.AssemblyLoadContext" />. Its value can be <see langword="null" />.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>