[MPlayer-dev-eng] [PATCH] Add mp_strings.c with mp_asprintf function.

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Feb 26 12:20:04 CET 2011


On Thu, Feb 24, 2011 at 01:09:28AM +0100, Clément Bœsch wrote:
> Patches updated.

Seems fine to me.

> -static int make_noauth_url(URL_t *url, char *dst, int dst_size)
> +static char *get_noauth_url(URL_t *url)
> -int make_http_proxy_url(URL_t *proxy, const char *host_url, char *dst,
> -                        int dst_size)
> +char *get_http_proxy_url(URL_t *proxy, const char *host_url)

Since you're changing it anyway, I think you could change
URL_t to const URL_t.
Doesn't matter much though.


More information about the MPlayer-dev-eng mailing list