CURL review request

jdrewsen jdrewsen at nospam.com
Tue Aug 16 12:55:51 PDT 2011


Den 16-08-2011 16:43, Alix Pexton skrev:
> On 16/08/2011 12:48, 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
>
> Spotted another issue...
>
> In the example for Protocol.onReceive, there is a mismatch between the
> parameter names specified for the delegate and those used inside it.

you mean this one:

client.onReceive = (ubyte[] data) { writeln("Got data", cast(char[]) 
data); return data.length;};

I cannot identify the problem?

> Also, all the XREF macros point to std_curl.html. instead of
> etc_curl.html, is this going to become std.curl when incorporated into
> Phobos?
>
> A...

A fix to the std ddoc config is needed.

As explained in reply to another post I believe this wrapper should be 
put in the etc package.

/Jonas



More information about the Digitalmars-d mailing list