std.path review: second update

[Lars T. Kyllingstad]

On Fri, 29 Jul 2011 18:06:58 +0000, Lars T. Kyllingstad wrote:

> Here's a new update based on your comments and requests.  [...]

You may have noticed that I did not incorporate Steve's suggestion to 
make the directory separator(s) a template parameter.  The reason is that 
this is the *smallest* difference between paths on the different 
platforms.  Drive letters and UNC paths are a much bigger difference, and 
make interoperability near impossible.

Therefore, I would just like to reiterate a suggestion I made on the 
Phobos list a while ago, which was then shot down.  What if we put the 
entire module inside a template, like this:

    enum Platform { windows, posix }
    enum CaseSensitive { yes, no }

    template Path(Platform platform, CaseSensitive caseSensitive)
    {
        /* Every function in the module goes inside this template,
           and instead of

               version (Windows) { ... }

           and so on, we use

               static if (platform == Platform.windows) { ... }

           That way, if people for some reason need to deal with
           POSIX paths on Windows, for instance, they can just write

               Path!(Platform.posix).someFunction(myPath);
        */
    }

    // Of course, we don't want to add a cumbersome prefix every time
    // we call a function for the current platform.  By mixing in the
    // template, std.path can be used *exactly* like now:

    version (Windows)    private enum currentPlatform = Platform.windows;
    else version (Posix) private enum currentPlatform = Platform.posix;

    mixin Path!(currentPlatform, platformDefaultCaseSensitivity);

What do you think?

-Lars