On Tue, Jun 14, 2011 at 4:09 PM, Rainer Schuetze <span dir="ltr"><<a href="mailto:r.sagitario@gmx.de" target="_blank">r.sagitario@gmx.de</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>On 08.06.2011 21:29, Lars Tandle Kyllingstad wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
As you may already know, I've had a new version of std.path in the works<br>
for a while.  It is ready now, and I am waiting for my turn in the<br>
review queue in the main NG.  In the meantime, I just thought I'd run it<br>
by this mailing list to iron out the worst wrinkles.<br>
<br>
So, if you feel like it, please comment.<br>
<br>
Code:<br>
<a href="https://github.com/kyllingstad/phobos/blob/std-path/std/path.d" target="_blank">https://github.com/kyllingstad/phobos/blob/std-path/std/path.d</a><br>
<br>
Docs:<br>
<a href="http://www.kyllingen.net/code/new-std-path/phobos-prerelease/std_path.html" target="_blank">http://www.kyllingen.net/code/new-std-path/phobos-prerelease/std_path.html</a><br>
<br>
<br>
-Lars<br>
</blockquote>
<br></div>
Looks good to me. A few notes:<br>
<br>
- I agree with Jose that toAbsolute and expandTilde feel a little out-of-place in this module as they are not only string manipulation functions.<br><br></blockquote><div>I think, the way phobos is organized (distinct std.file and std.path modules) makes this problem inevitable.</div>


<div><font face="Arial, Verdana, sans-serif"><span style="line-height:18px">The only solutions are to either keep them in std.path (simply as exceptions to the no-performing-IO rule) or combine std.path and std.file (which is probably a worse idea).</span></font></div>


</div>