style guide fixes - PR 4/8 (dotnet#1384)

* style guide fixes

* Apply suggestions from PR review

Co-Authored-By: mairaw <[email protected]>

* feedback

Author: mairaw

xml/System.Activities.Presentation.Model/Change.xml

@@ -87,7 +87,7 @@
       </ReturnValue>
       <Docs>
         <summary>When implemented in a derived class, gets a description of the change.</summary>
-        <value>Returns a <see cref="T:System.String" /> that contains a description of the change.</value>
+        <value>A string that contains a description of the change.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -109,7 +109,7 @@
       <Parameters />
       <Docs>
         <summary>When implemented in a derived class, retrieves the inverse of the change.</summary>
-        <returns>Returns a <see cref="T:System.Activities.Presentation.Model.Change" /> that contains the inverse of the current instance.</returns>
+        <returns>The inverse of the current instance.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System.Activities.Presentation.Model/EditingScope.xml

@@ -70,7 +70,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets a list of changes.</summary>
-        <value>Returns a <see cref="T:System.Collections.Generic.List`1" /> that contains the changes.</value>
+        <value>A list of changes.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System.Activities.Presentation.Model/ModelFactory.xml

@@ -50,7 +50,7 @@
         <param name="context">The designer's editing context.</param>
         <param name="item">The <see cref="T:System.Activities.Presentation.Model.ModelItem" /> to clone.</param>
         <summary>Creates a new <see cref="T:System.Activities.Presentation.Model.ModelItem" /> by creating a deep copy of the instance provided.</summary>
-        <returns>Returns the newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
+        <returns>The newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -90,7 +90,7 @@
         <param name="itemType">The type of the <see cref="T:System.Activities.Presentation.Model.ModelItem" /> to create.</param>
         <param name="arguments">An optional array of arguments that should be passed to the constructor of the item.</param>
         <summary>Creates a new <see cref="T:System.Activities.Presentation.Model.ModelItem" /> for the given item type.</summary>
-        <returns>Returns the newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
+        <returns>The newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -127,7 +127,7 @@
         <param name="options">A set of options used when creating the item.</param>
         <param name="arguments">An optional array of arguments that should be passed to the constructor of the item.</param>
         <summary>Creates a new <see cref="T:System.Activities.Presentation.Model.ModelItem" /> for the given item type.</summary>
-        <returns>Returns the newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
+        <returns>The newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -155,7 +155,7 @@
         <param name="type">The type that contains the static member being referenced.</param>
         <param name="memberName">The name of the static member being referenced.</param>
         <summary>Creates a new <see cref="T:System.Activities.Presentation.Model.ModelItem" /> that represents the value of a static member of the given class.</summary>
-        <returns>Returns the newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
+        <returns>The newly created <see cref="T:System.Activities.Presentation.Model.ModelItem" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System.Activities.Presentation.Model/ModelItemCollection.xml

@@ -312,7 +312,7 @@
       </ReturnValue>
       <Docs>
         <summary>Returns the count of items in the collection.</summary>
-        <value>Returns the <see cref="T:System.Int32" /> count of items in the collection.</value>
+        <value>The count of items in the collection.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   
@@ -344,7 +344,7 @@
       <Parameters />
       <Docs>
         <summary>Gets an enumerator that iterates through the <see cref="T:System.Activities.Presentation.Model.ModelItemCollection" />.</summary>
-        <returns>Returns an <see cref="T:System.Collections.Generic.IEnumerator`1" /> for the <see cref="T:System.Activities.Presentation.Model.ModelItemCollection" />.</returns>
+        <returns>An enumerator for the <see cref="T:System.Activities.Presentation.Model.ModelItemCollection" />.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -723,7 +723,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets an object that can be used to synchronize access to the collection.</summary>
-        <value>Returns an <see cref="T:System.Object" />.</value>
+        <value>An object that can be used to synchronize access to the collection.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -779,7 +779,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets the number of elements contained in the collection.</summary>
-        <value>Returns an <see cref="T:System.Int32" /> value that is the number of elements contained in the collection.</value>
+        <value>The number of elements contained in the collection.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -838,7 +838,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets an object that can be used to synchronize access to the <see cref="T:System.Collections.ICollection" />.</summary>
-        <value>Returns an <see cref="T:System.Object" /> that can be used to synchronize access to the <see cref="T:System.Collections.ICollection" />.</value>
+        <value>An object that can be used to synchronize access to the <see cref="T:System.Collections.ICollection" />.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -870,7 +870,7 @@
       <Parameters />
       <Docs>
         <summary>Returns an enumerator that iterates through a collection.</summary>
-        <returns>Returns an <see cref="T:System.Collections.IEnumerator" /> that can be used to iterate through the collection.</returns>
+        <returns>An enumerator that can be used to iterate through the collection.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -897,7 +897,7 @@
       <Docs>
         <param name="value">The object to add to the collection.</param>
         <summary>Adds a model item to the end of the <see cref="T:System.Activities.Presentation.Model.ModelItemCollection" />.</summary>
-        <returns>Returns the index of the newly added object.</returns>
+        <returns>The index of the newly added object.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -987,7 +987,7 @@
       <Docs>
         <param name="value">The <see cref="T:System.Object" /> to locate in the <see cref="T:System.Collections.IList" />.</param>
         <summary>Determines the index of a specific item in the <see cref="T:System.Collections.IList" />.</summary>
-        <returns>Returns the <see cref="T:System.Int32" /> index of <paramref name="value" /> if found in the list; otherwise, -1.</returns>
+        <returns>The index of <paramref name="value" /> if found in the list; otherwise, -1.</returns>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>
@@ -1177,4 +1177,4 @@
       </Docs>
     </Member>
   </Members>
-</Type>
+</Type>

xml/System.Activities.Presentation.Model/ModelItemDictionary.xml

@@ -133,7 +133,7 @@
         <param name="key">The key in the key/value pair to be added.</param>
         <param name="value">The value in the key/value pair to be added.</param>
         <summary>Adds the value to the dictionary under the given key. This method wraps the key and value in a set of <see cref="T:System.Activities.Presentation.Model.ModelItem" /> objects.</summary>
-        <returns>Returns a <see cref="T:System.Activities.Presentation.Model.ModelItem" /> that represents the key.</returns>
+        <returns>A <see cref="T:System.Activities.Presentation.Model.ModelItem" /> that represents the key.</returns>
         <remarks>To be added.</remarks>
         <exception cref="T:System.ArgumentNullException">If <paramref name="key" /> or <paramref name="value" /> is <see langword="null" />.</exception>
         <exception cref="T:System.InvalidOperationException">If the <see cref="T:System.Activities.Presentation.Model.ModelItemDictionary" /> is read-only.</exception>

xml/System.IdentityModel.Tokens/Saml2NameIdentifier.xml

@@ -113,7 +113,7 @@
         <ReturnType>System.IdentityModel.Tokens.EncryptingCredentials</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Get or sets the credentials used for encrypting the name identifier in an EncryptedID element.</summary>
+        <summary>Gets or sets the credentials used for encrypting the name identifier in an EncryptedID element.</summary>
         <value>A <see cref="T:System.IdentityModel.Tokens.EncryptingCredentials" /> that represents the credentials used to encrypt the name identifier.</value>
         <remarks>To be added.</remarks>
       </Docs>

xml/System.IdentityModel.Tokens/Saml2SubjectConfirmationData.xml

@@ -117,7 +117,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets the collection of cryptographic keys that are used in some way to authenticate an attesting entity. [Saml2Core, 2.4.1.3]</summary>
-        <value>Returns a collection of type <see cref="T:System.IdentityModel.Tokens.SecurityKeyIdentifier" /> that contains the keys.</value>
+        <value>A collection of type <see cref="T:System.IdentityModel.Tokens.SecurityKeyIdentifier" /> that contains the keys.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   

xml/System.IdentityModel.Tokens/SamlAction.xml

@@ -164,7 +164,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets or sets the action that the subject of the <see cref="T:System.IdentityModel.Tokens.SamlAuthorizationDecisionStatement" /> can perform on the specified resource.</summary>
-        <value>Gets or sets the action that the subject of the <see cref="T:System.IdentityModel.Tokens.SamlSecurityToken" /> security token seeks to perform on the specified resource.</value>
+        <value>The action that the subject of the <see cref="T:System.IdentityModel.Tokens.SamlSecurityToken" /> security token seeks to perform on the specified resource.</value>
         <remarks>
           <format type="text/markdown"><![CDATA[  
   

xml/System.Linq.Expressions/BinaryExpression.xml

@@ -188,7 +188,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets a value that indicates whether the expression tree node can be reduced.</summary>
-        <value>True if the expression tree node can be reduced, otherwise false.</value>
+        <value><see langword="true" /> if the expression tree node can be reduced; otherwise, <see langword="false" />.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>

xml/System.Linq.Expressions/DebugInfoExpression.xml

@@ -218,7 +218,7 @@
       </ReturnValue>
       <Docs>
         <summary>Gets the value to indicate if the <see cref="T:System.Linq.Expressions.DebugInfoExpression" /> is for clearing a sequence point.</summary>
-        <value>True if the <see cref="T:System.Linq.Expressions.DebugInfoExpression" /> is for clearing a sequence point, otherwise false.</value>
+        <value><see langword="true" /> if the <see cref="T:System.Linq.Expressions.DebugInfoExpression" /> is for clearing a sequence point; otherwise, <see langword="false" />.</value>
         <remarks>To be added.</remarks>
       </Docs>
     </Member>