[FFmpeg-devel] [PATCH/RFC] Prefer getaddrinfo over gethostbyname

Martin Storsjö martin
Mon Jan 11 08:47:06 CET 2010


On Fri, 8 Jan 2010, M?ns Rullg?rd wrote:

> Martin Storsj? <martin at martin.st> writes:

> >> >      type=$2
> >> > +    type_quoted=$(echo "$type" | sed 's/ /_/g')
> >> >      shift 2
> >> >      disable $type
> >> 
> >> You missed one.
> >
> > Missed one of what?
> 
> The disable line.

Ah, good catch.

> >> Use enable_safe/disable_safe instead.  They do the required
> >> replacements already.
> >
> > No, they don't escape spaces at the moment:
> >
> >> enable_safe(){
> >>     enable $(echo "$*" | sed 's/[^A-Za-z0-9_ ]/_/g')
> >> }
> >
> > And if they were made to do that, enabling more than one item at a time 
> > would break.
> 
> Right... that can be fixed though.  Give me a moment.

Ping, any progress on this?

// Martin



More information about the ffmpeg-devel mailing list