From 76263fe26fcb3e5a3a76a72e834ae200337b5036 Mon Sep 17 00:00:00 2001
From: Maira Wenzel <mairaw@microsoft.com>
Date: Wed, 11 Dec 2019 12:09:56 -0800
Subject: [PATCH 1/2] improve language

---
 .../ObjectItemCollection.xml                       | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml b/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml
index 12756bd55b1..d517a811d65 100644
--- a/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml
+++ b/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml
@@ -274,10 +274,10 @@
         <Parameter Name="clrType" Type="System.Type" RefType="out" Index="1" FrameworkAlternate="netframework-4.5;netframework-4.5.1;netframework-4.5.2;netframework-4.6;netframework-4.6.1;netframework-4.6.2;netframework-4.7;netframework-4.7.1;netframework-4.7.2;netframework-4.8" />
       </Parameters>
       <Docs>
-        <param name="objectSpaceType">The OSpace enum type to look up</param>
-        <param name="clrType">The CLR enum type of the OSpace argument</param>
-        <summary>Returns the underlying CLR type for the specified OSpace enum type argument. If the DataSpace of the parameter is not OSpace, the method returns false and sets the out parameter to null.</summary>
-        <returns><see langword="true" /> on success, false on failure</returns>
+        <param name="objectSpaceType">The OSpace enum type to look up.</param>
+        <param name="clrType">When this method returns, contains the CLR enum type of the OSpace argument.</param>
+        <summary>Returns the underlying CLR type for the specified OSpace enum type argument.</summary>
+        <returns><see langword="true" /> if the CLR type can be obtained; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -303,10 +303,10 @@
       </Parameters>
       <Docs>
         <param name="objectSpaceType">A <see cref="T:System.Data.Metadata.Edm.StructuralType" /> that represents the object space type.</param>
-        <param name="clrType">The CLR type.</param>
-        <summary>Returns a CLR type corresponding to the <see cref="T:System.Data.Metadata.Edm.StructuralType" /> supplied by the objectSpaceType parameter.</summary>
+        <param name="clrType">When this method returns, contains the CLR enum type of the OSpace argument.</param>
+        <returns><see langword="true" /> if the CLR type can be obtained; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
         <returns>
-          <see langword="true" /> if there is a type that matches the search criteria; otherwise, <see langword="false" />.</returns>
+          <see langword="true" /> if there is a type that matches the search criteria; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

From fdee3ef65633013ee5bf190377bd91bf9043185b Mon Sep 17 00:00:00 2001
From: Maira Wenzel <mairaw@microsoft.com>
Date: Mon, 16 Dec 2019 23:08:38 -0800
Subject: [PATCH 2/2] feedback

---
 xml/System.Data.Metadata.Edm/ObjectItemCollection.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml b/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml
index d517a811d65..de17765deb9 100644
--- a/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml
+++ b/xml/System.Data.Metadata.Edm/ObjectItemCollection.xml
@@ -277,7 +277,8 @@
         <param name="objectSpaceType">The OSpace enum type to look up.</param>
         <param name="clrType">When this method returns, contains the CLR enum type of the OSpace argument.</param>
         <summary>Returns the underlying CLR type for the specified OSpace enum type argument.</summary>
-        <returns><see langword="true" /> if the CLR type can be obtained; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
+        <returns>
+          <see langword="true" /> if the CLR type can be obtained; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -304,9 +305,8 @@
       <Docs>
         <param name="objectSpaceType">A <see cref="T:System.Data.Metadata.Edm.StructuralType" /> that represents the object space type.</param>
         <param name="clrType">When this method returns, contains the CLR enum type of the OSpace argument.</param>
-        <returns><see langword="true" /> if the CLR type can be obtained; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
         <returns>
-          <see langword="true" /> if there is a type that matches the search criteria; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
+          <see langword="true" /> if the CLR type can be obtained; otherwise, <see langword="false" />. If the DataSpace property of the parameter is not <see cref="F:System.Data.Metadata.Edm.DataSpace.OSpace" />, the method returns <see langword="false" /> and sets the <paramref name="clrType" /> parameter to <see langword="null" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>