[phobos] RFC: std.path

[Jimmy Cao]

On Tue, Jun 14, 2011 at 4:09 PM, Rainer Schuetze <r.sagitario at gmx.de> wrote:

> On 08.06.2011 21:29, Lars Tandle Kyllingstad wrote:
>
>> As you may already know, I've had a new version of std.path in the works
>> for a while.  It is ready now, and I am waiting for my turn in the
>> review queue in the main NG.  In the meantime, I just thought I'd run it
>> by this mailing list to iron out the worst wrinkles.
>>
>> So, if you feel like it, please comment.
>>
>> Code:
>> https://github.com/kyllingstad/phobos/blob/std-path/std/path.d
>>
>> Docs:
>> http://www.kyllingen.net/code/new-std-path/phobos-prerelease/std_path.html
>>
>>
>> -Lars
>>
>
> Looks good to me. A few notes:
>
> - 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.
>
> I think, the way phobos is organized (distinct std.file and std.path
modules) makes this problem inevitable.
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).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.puremagic.com/pipermail/phobos/attachments/20110614/b0311aed/attachment.html>