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

Clément Bœsch ubitux at gmail.com
Sat Feb 26 13:03:20 CET 2011


On Sat, Feb 26, 2011 at 12:20:04PM +0100, Reimar Döffinger wrote:
> On Thu, Feb 24, 2011 at 01:09:28AM +0100, Clément Bœsch wrote:
> > Patches updated.
> 
> Seems fine to me.
> 

Thanks, applied.

> > -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.

Applied in a different commit.

-- 
Clément B.


More information about the MPlayer-dev-eng mailing list