Hi Daniel,

> I would strongly recommend using "./configure --enable-debug" instead.
> Simpler
> to write, adds many more pendantic compiler options and runs 'make
> checksrc'
> by default on the code etc.

Thanks! I had assumed that option was for debug symbols or preprocessor 
definitions.

> If the configure check wrongly detects your strdup as present when it is fact
> isn't, then CURL_CHECK_FUNC_STRDUP in m4/curl-functions.m4 is bad.
> Which is
> surprising to me since it has been around unmodified for a very long time
> now.
> The test seems rather complete to me...

Thanks for the pointer. tst_compi_strdup  seems incomplete. According to the 
documentation of AC_LANG_PROGRAM, it puts the second argument into a main 
function. Calling strdup inside a function works fine and only results in 
warnings. What doesn't work is referencing it at the file scope, which is what 
easy.c does:
curl_strdup_callback Curl_cstrdup = (curl_strdup_callback)system_strdup;

Probably noone tried to compile with -std=c90 yet, or they fixed curl_config.h 
manually.

Marcel


-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette:   https://curl.haxx.se/mail/etiquette.html

Reply via email to