cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] unit test for Curl_llist_insert_next

From: Amr Shahin <amrnablus_at_gmail.com>
Date: Sat, 22 Jan 2011 23:51:34 +0200

On Fri, Jan 21, 2011 at 12:19 AM, Daniel Stenberg <daniel_at_haxx.se> wrote:

> On Thu, 20 Jan 2011, Amr Shahin wrote:
>
> the attached file contains a patch that will add a couple of test cases to
>> Curl_llist_insert_next, hopefully it covers all the cases.
>>
>> let me know what you think.
>>
>
> Thanks a lot! My thoughts on the patch:
>
> 1 - you used too long lines, we have a 80 column limit
>
> 2 - make sure the tests all run fine before you submit your patch
>
> I fixed those issues (this time) and committed.
>
> --
>
> / daniel.haxx.se
> -------------------------------------------------------------------
> List admin: http://cool.haxx.se/list/listinfo/curl-library
> Etiquette: http://curl.haxx.se/mail/etiquette.html
>

Thanks, i'll be working on the rest of llist functions.

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2011-01-22