<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">
<META NAME="Generator" CONTENT="MS Exchange Server version 6.5.7638.1">
<TITLE>RE: [Mono-dev] HttpRequest.MapPath and UrlUtils.Combine codeduplication</TITLE>
</HEAD>
<BODY>
<!-- Converted from text/plain format -->

<P><FONT SIZE=2>Hello Gonzalo,<BR>
<BR>
Probably I had to explain more. If you follow the flow in HttpRequest.MapPath, then you see that after handling backslashes, function UrlUtils.Combine is called, and the latter already has the very same code. First backslashes are handled, then check for rooted and call to Canonic. The only difference was that Combine checked for backslashes like the following:<BR>
<BR>
relPath = relPath.Replace (&quot;\\&quot;, &quot;/&quot;);<BR>
<BR>
So I replaced it with code from MapPath, looking like:<BR>
<BR>
if (System.IO.Path.DirectorySeparatorChar !='/')<BR>
&nbsp;&nbsp; relPath = relPath.Replace (System.IO.Path.DirectorySeparatorChar, '/');<BR>
<BR>
If you say that you want to replace backslashes on non-windows system, too, then I will leave Combine intact. But anyway the code that I removed from MapPath can be removed safely, Combine does all that anyway.<BR>
<BR>
What do you think?<BR>
<BR>
Regards,<BR>
Andrew.<BR>
<BR>
<BR>
-----Original Message-----<BR>
From: mono-devel-list-bounces@lists.ximian.com on behalf of Gonzalo Paniagua Javier<BR>
Sent: Sun 4/9/2006 10:56 PM<BR>
To: mono-devel-list@lists.ximian.com<BR>
Subject: Re: [Mono-dev] HttpRequest.MapPath and UrlUtils.Combine codeduplication<BR>
<BR>
On Sun, 2006-04-09 at 08:08 -0700, Andrew Skiba wrote:<BR>
&gt; Hello.<BR>
&gt;<BR>
&gt; I found code duplication, please review the patch to remove it.<BR>
<BR>
Sometimes we have ASP.NET files copied from windows that have stuff like<BR>
'Src=&quot;.\someotherfile.ascx&quot;. The code that you're removing handles that.<BR>
Don't commit.<BR>
<BR>
-Gonzalo<BR>
<BR>
<BR>
_______________________________________________<BR>
Mono-devel-list mailing list<BR>
Mono-devel-list@lists.ximian.com<BR>
<A HREF="http://lists.ximian.com/mailman/listinfo/mono-devel-list">http://lists.ximian.com/mailman/listinfo/mono-devel-list</A><BR>
<BR>
</FONT>
</P>

</BODY>
</HTML>