On Wed, Aug 17, 2011 at 10:41 PM, dsimcha <dsimcha@yahoo.com> wrote:
On 8/16/2011 7:48 AM, Jonas Drewsen wrote:
Hi all,

This is a review request for the curl wrapper. Please read the "known
issues" in the top of the source file and if possible suggest a solution.

We also need somebody for running the review process. Anyone?

Code:
https://github.com/jcd/phobos/blob/curl-wrapper/etc/curl.d
Docs:
http://freeze.steamwinter.com/D/web/phobos/etc_curl.html

Demolish!

/Jonas

One more comment:  In various places, you have someFunction(in const(char)[] someString).  Const and in are redundant.  In is synonymous with const.

Also, anyone have any luck getting Curl libs to work with DMD on Windows?  The binaries available at http://curl.haxx.se/download.html look like they're COFF.  (<rant> Why do we still use freakin' OMF? </rant>)  Also, the source doesn't look like it's pleasant to compile.

I've always been using etc.c.curl on Windows.  Use implib to generate .lib files from the dlls.