On Tue, 17 Mar 2015, Stefan Bühler wrote:

It would be nice to get feedback for this specific commit, and maybe for the others as well.

Please run lib/checksrc.pl on the code after your changes and correct the warnings. There are a few style nits.

Have you given any thoughts on if it is possible to add any unit tests that verify this functionality?

--

 / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to