On Tue, Jun 14, 2011 at 4:09 PM, Rainer Schuetze <[email protected]> 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).
_______________________________________________ phobos mailing list [email protected] http://lists.puremagic.com/mailman/listinfo/phobos
