Daniel Stenberg <daniel@xxxxxxx> writes: > On Mon, 5 Sep 2011, Jonathan Nieder wrote: > >> + if (!curl_errorstr[0]) >> + strlcpy(curl_errorstr, >> + curl_easy_strerror(results.curl_result), >> + sizeof(curl_errorstr)); > > (as libcurl hacker) > > For the record, in libcurl we work on providing "extra" information in > the error buffer when there is additional info to provide that would > help. In some cases we deem there isn't (or we just to provide any) > and then the generic error message is good enough and could indeed be > used like this... Sorry if I am a bit slow but are you saying: (1) we provide "extra" but your patch is not using it which is bad? (2) the above is Ok but there are better ways to do it? (3) something else? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html