On Thu, 2 Aug 2018, Geoff Beier wrote:

The setters would be important to us. I might be bikeshedding here, but the ability to add to the query would be very nice. So something like curl_url_query_append(urlp, "numitems", 3)

That ("numitems", 3) approach is very specific for adding a "[name]=[number]" snippet though. Shouldn't it rather be a function for appdending a generic string?

  curl_url_query_append(urp, "numitems=3");

... or using the alternative B API (https://github.com/curl/curl/wiki/URL-API#url-api-alternative-b) approach:

  curl_url_append(urp, CURLUPART_QUERY, "numitems=3");

An unsigned type feels more appropriate for port than int does.

Yes, or as a full blown string to make it a generic part in alternative B.

I assume this is obvious, but if this API gets added to curl, a setopt that takes a CURLURL would be wanted :)

Yes!

--

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

Reply via email to