On Fri, Jan 21, 2011 at 12:19 AM, Daniel Stenberg <[email protected]> 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

Reply via email to