Hello again.

After posting the previous patch I noticed that UrlEncodeUnicode has all
the problems that UrlEncode had, plus it did not handle right all
characters after 0x100. So I made some refactoring, united the common
code into private method UrlEncodeChar, and made additions to the unit
test so it checks the both. Please review the new patch.

If nobody objects, I will commit.

Thank you.
Andrew.

> -----Original Message-----
> From: [EMAIL PROTECTED] 
> [mailto:[EMAIL PROTECTED] On Behalf 
> Of Andrew Skiba
> Sent: Sunday, April 02, 2006 14:09
> To: Ben Maurer; [EMAIL PROTECTED]
> Cc: Mono-devel-list@lists.ximian.com
> Subject: [Mono-dev] UrlEncode patch
> 
> Hi guys,
> 
> The patch contains new unittest, passing on dotnet and 
> failing on mono, and fix for that unittest.
> 
> This is an old issue, probably you remember as Ben suggested 
> an improvement for the testcase. I did that improvement, 
> please tell me what you think now. The new unittest passes on 
> dotnet and after the fix it passes on mono, too.
> 
> Thank you.
> Andrew.
> 

Attachment: HttpUtility.patch
Description: HttpUtility.patch

_______________________________________________
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list

Reply via email to