Skip to content
This repository was archived by the owner on Nov 20, 2018. It is now read-only.

Commit 825b1e4

Browse files
committed
Review feedback 3
1 parent b707ea7 commit 825b1e4

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

src/Microsoft.Extensions.WebEncoders.Core/UrlPathDecoder.cs

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,16 @@ public class UrlPathDecoder
1919
/// <summary>
2020
/// Unescape a url char array in place. Returns the length of the result.
2121
///
22-
/// - Everything is unescaped except %2F ('\')
22+
/// - Everything is unescaped except %2F ('/')
2323
/// - UTF8 bytes are tested for formatting, overlong encoding, surrogates and value ranges
2424
/// - Invalid escaped sequence are copied to output as is
2525
/// - It doesn't check if the string contains query
2626
/// </summary>
27+
/// <param name="buffer">
28+
/// The char array contains sequence of charactors to be decoded. The
29+
/// result will be saved in the same array.
30+
/// </param>
31+
/// <returns>The length of the result.</returns>
2732
public static int DecodeInPlace(char[] buffer)
2833
{
2934
if (buffer == null)
@@ -42,11 +47,13 @@ public static int DecodeInPlace(char[] buffer)
4247
/// <summary>
4348
/// Unescape a url path string.
4449
///
45-
/// - Everything is unescaped except %2F ('\')
50+
/// - Everything is unescaped except %2F ('/')
4651
/// - UTF8 bytes are tested for formatting, overlong encoding, surrogates and value ranges
4752
/// - Invalid escaped sequence are copied to output as is
4853
/// - It doesn't check if the string contains query
4954
/// </summary>
55+
/// <param name="original">The string to be decoded.</param>
56+
/// <returns>The decoded result.</returns>
5057
public static string Decode(string original)
5158
{
5259
if (original == null)

0 commit comments

Comments
 (0)