curl-library
Re: [PATCH] unit test for Curl_llist_insert_next
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