cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] unit test for Curl_llist_insert_next

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Thu, 20 Jan 2011 23:19:03 +0100 (CET)

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
Received on 2011-01-20