Hi Daniel, I see from the above commit that the curl command line tool now has tool_strdup.c|h rather than using strdup.c|h from curl.
As such I need to fix up generate.bat as the few files that are included from libcurl are currently hard coded in there (as I don't parse the makefiles in this batch file as it is intended for use on Windows when working directly from the repo). However, I was wondering.. Should the new Curl_memdup() function be located in a new curl_memdup.c|h (like curl_memchr.c|h) or even curl_memory.c instead rather than having two versions of the strdup() function? This would mean that: * Curl_tool could share strdup.c|h again and avoid code duplication * Curl_memdup() would fit with Curl_memchr() and the memory definitions in curl_memory.h * I don't have to fix update generate.bat - Although I've already done that ready to push so a very minor reason ;-) Kind Regards Steve ------------------------------------------------------------------- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
