system.io.compression fixups

Author: gewarren

xml/System.IO.Compression/ZipArchive.xml

@@ -385,55 +385,56 @@ If the comment byte length is larger than <xref:System.UInt16.MaxValue>, it will
       </Parameters>
       <Docs>
         <param name="stream">The input or output stream.</param>
-        <param name="mode">See the description of the ZipArchiveMode enum. Read requires the stream to support reading, Create requires the stream to support writing, and Update requires the stream to support reading, writing, and seeking.</param>
+        <param name="mode">One of the enumeration values that specifies whether the stream supports reading, writing, and seeking.</param>
         <param name="leaveOpen">
           <see langword="true" /> to leave the stream open upon disposing the ZipArchive, otherwise <see langword="false" />.</param>
         <param name="entryNameEncoding">
-          <para>The encoding to use when reading or writing entry names and comments in this ZipArchive.</para>
-          <para>NOTE: Specifying this parameter to values other than <see langword="null" /> is discouraged.</para>
-          <para>However, this may be necessary for interoperability with ZIP archive tools and libraries that do not correctly support UTF-8 encoding for entry names.&lt;br /&gt; This value is used as follows:</para>
+          The encoding to use when reading or writing entry names and comments in this ZipArchive.
+        </param>
+        <param name="cancellationToken">The optional cancellation token to monitor.</param>
+        <summary>Asynchronously initializes and returns a new instance of <see cref="T:System.IO.Compression.ZipArchive" /> on the given stream in the specified mode, specifying whether to leave the stream open, with an optional encoding and an optional cancellation token.</summary>
+        <returns>To be added.</returns>
+        <remarks>
+        <para>Specifying a value for <paramref name="entryNameEncoding" /> other than <see langword="null" /> is discouraged. However, this might be necessary for interoperability with ZIP archive tools and libraries that don't correctly support UTF-8 encoding for entry names.</para>
+        <para>This value is used as follows:</para>
           <para><strong>Reading (opening) ZIP archive files:</strong></para>
-          <para>If <c>entryNameEncoding</c> is not specified (<c>== null</c>):</para>
+          <para>If <paramref name="entryNameEncoding" /> is not specified (<c>== null</c>):</para>
             <list>
-              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header is <em>not</em> set, use the current system default code page (<c>Encoding.Default</c>) in order to decode the entry name and comment.</item>
-              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header <em>is</em> set, use UTF-8 (<c>Encoding.UTF8</c>) in order to decode the entry name and comment.</item>
+              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header is <em>not</em> set, use the current system default code page (<c>Encoding.Default</c>) to decode the entry name and comment.</item>
+              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header <em>is</em> set, use UTF-8 (<c>Encoding.UTF8</c>) to decode the entry name and comment.</item>
             </list>
-            <para>If <c>entryNameEncoding</c> is specified (<c>!= null</c>):</para>
+            <para>If <paramref name="entryNameEncoding" /> is specified (<c>!= null</c>):</para>
             <list>
-              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header is <em>not</em> set, use the specified <c>entryNameEncoding</c> in order to decode the entry name and comment.</item>
-              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header <em>is</em> set, use UTF-8 (<c>Encoding.UTF8</c>) in order to decode the entry name and comment.</item>
+              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header is <em>not</em> set, use the specified <paramref name="entryNameEncoding" /> to decode the entry name and comment.</item>
+              <item>For entries where the language encoding flag (EFS) in the general purpose bit flag of the local file header <em>is</em> set, use UTF-8 (<c>Encoding.UTF8</c>) to decode the entry name and comment.</item>
             </list>
             <para><strong>Writing (saving) ZIP archive files:</strong> </para>
-          <para>If <c>entryNameEncoding</c> is not specified (<c>== null</c>):</para>
+          <para>If <paramref name="entryNameEncoding" /> is not specified (<c>== null</c>):</para>
           <list>
-            <item>For entry names and comments that contain characters outside the ASCII range, the language encoding flag (EFS) will be set in the general purpose bit flag of the local file header, and UTF-8 (<c>Encoding.UTF8</c>) will be used in order to encode the entry name and comment into bytes.</item>
-            <item>For entry names and comments that do not contain characters outside the ASCII range, the language encoding flag (EFS) will not be set in the general purpose bit flag of the local file header, and the current system default code page (<c>Encoding.Default</c>) will be used to encode the entry names and comments into bytes.</item>
+            <item>For entry names and comments that contain characters outside the ASCII range, the language encoding flag (EFS) will be set in the general purpose bit flag of the local file header, and UTF-8 (<c>Encoding.UTF8</c>) will be used to encode the entry name and comment into bytes.</item>
+            <item>For entry names and comments that don't contain characters outside the ASCII range, the language encoding flag (EFS) will not be set in the general purpose bit flag of the local file header, and the current system default code page (<c>Encoding.Default</c>) will be used to encode the entry names and comments into bytes.</item>
           </list>
-          <para>If <c>entryNameEncoding</c> is specified (<c>!= null</c>):</para>
+          <para>If <paramref name="entryNameEncoding" /> is specified (<c>!= null</c>):</para>
           <list>
-            <item>The specified <c>entryNameEncoding</c> will always be used to encode the entry names and comments into bytes.</item>
+            <item>The specified <paramref name="entryNameEncoding" /> will always be used to encode the entry names and comments into bytes.</item>
           </list>
-          <para>The language encoding flag (EFS) in the general purpose bit flag of the local file header will be set if and only if the specified <c>entryNameEncoding</c> is a UTF-8 encoding.</para>
-        </param>
-        <param name="cancellationToken">The optional cancellation token to monitor.</param>
-        <summary>Asynchronously initializes and returns a new instance of <see cref="T:System.IO.Compression.ZipArchive" /> on the given stream in the specified mode, specifying whether to leave the stream open, with an optional encoding and an optional cancellation token.</summary>
-        <returns>To be added.</returns>
-        <remarks>To be added.</remarks>
+          <para>The language encoding flag (EFS) in the general purpose bit flag of the local file header will be set if and only if the specified <paramref name="entryNameEncoding" /> is a UTF-8 encoding.</para>
+        </remarks>
         <exception cref="T:System.ArgumentException">
           <para>The stream is already closed.</para>
           <para>-or-</para>
-          <para>mode is incompatible with the capabilities of the stream.</para>
+          <para><paramref name="mode" /> is incompatible with the capabilities of the stream.</para>
         </exception>
         <exception cref="T:System.ArgumentNullException">The stream is <see langword="null" />.</exception>
-        <exception cref="T:System.ArgumentOutOfRangeException">mode specified an invalid value.</exception>
+        <exception cref="T:System.ArgumentOutOfRangeException"><paramref name="mode" /> specified an invalid value.</exception>
         <exception cref="T:System.IO.InvalidDataException">
-          <para>The contents of the stream could not be interpreted as a Zip file.</para>
+          <para>The contents of the stream could not be interpreted as a ZIP file.</para>
           <para>-or-</para>
-          <para>mode is Update and an entry is missing from the archive or is corrupt and cannot be read.</para>
+          <para><paramref name="mode" /> is Update and an entry is missing from the archive or is corrupt and cannot be read.</para>
           <para>-or-</para>
-          <para>mode is Update and an entry is too large to fit into memory.</para>
+          <para><paramref name="mode" /> is Update and an entry is too large to fit into memory.</para>
         </exception>
-        <exception cref="T:System.ArgumentException">If a Unicode encoding other than UTF-8 is specified for the <c>entryNameEncoding</c>.</exception>
+        <exception cref="T:System.ArgumentException">A Unicode encoding other than UTF-8 was specified for <paramref name="entryNameEncoding" />.</exception>
       </Docs>
     </Member>
     <MemberGroup MemberName="CreateEntry">