Curl wrapper review

[Bernard Helyer]

I'll just post my thoughts here while they're fresh. It looks good. The 
documentation is what I'd expect from a Phobos module, as is the naming 
convention.

auto _basicFtp(T)(const(char)[] url, const(void)[] sendData, Ftp client)

If you don't want people using it, shouldn't it be marked private instead 
of using the underscore for obscurity?

private struct Pool(DATA) 
{
private:

You've marked private things as 'private foo;' everywhere else in the 
module, what's with the switch in styles for this struct? Also, as the 
whole struct is module private I'm not sure of the utility of marking 
members private. I guess it's a form of documentation.

But really, I'm grasping at straws. Even if the above were to remain, I 
would love to see this in Phobos yesterday. :)

-Bernard.