Jeff King <p...@peff.net> writes:

> We could even hide the whole thing away with something like:
>
>   void http_set_range(CURL *curl, long lo, long hi)
>   {
>       char buf[128];
>       int len = 0;
>
>       if (lo >= 0)
>               len += xsnprintf(buf + len, "%ld", lo);
>       len += xsnprintf(buf + len, "-");
>       if (hi >= 0)
>               len += xsnprintf(buf + len, "%ld", hi);
>
>       curl_easy_setopt(curl, CURLOPT_RANGE, buf);
>   }
>
> That would also make it easier to replace if we do need to keep an
> #ifdef for older versions of curl. But maybe it is just
> over-engineering.

I personally do not think this is an over-engineered version.  This
is exactly what I had in mind when I alluded to a small helper ;-)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to