Bugs item #3077599, was opened at 2010-09-28 23:41
Message generated for change (Comment added) made by bagder
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=3077599&group_id=976
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: libcurl
Group: portability problem
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Jeff A. Marr (jamarr81)
Assigned to: Daniel Stenberg (bagder)
Summary: Add curl_easy_setoptv
Initial Comment:
I am writing a small c++ wrapper for curl and came across an issue with curl_easy_setopt. There is no va_list alternative version of this function making it impossible to encapsulate; see: http://c-faq.com/varargs/handoff.html. To fix this we simply create a va_list version of this function: curl_easy_setoptv. I have attached a very small patch to fix this issue.
----------------------------------------------------------------------
>Comment By: Daniel Stenberg (bagder)
Date: 2010-09-28 23:52
Message:
It is not impossible at all. In fact it's even fairly easy. See the
src/main.c:_my_setopt() function for how we do it within the curl tool
source code.
Your function does make it easier but with the fancy type checks we have
for curl_easy_setopt() now and the fact that we can easily work around this
little flaw, I'm not at all convinced we should add this new function you
suggest.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=3077599&group_id=976
Received on 2010-09-28