[FFmpeg-devel] [PATCH] socklen_t definition
Diego Biurrun
diego
Tue Jan 1 23:54:57 CET 2008
On Tue, Jan 01, 2008 at 11:00:42AM -0500, Rich Felker wrote:
> On Tue, Jan 01, 2008 at 11:00:14AM +0000, M?ns Rullg?rd wrote:
> > Rich Felker <dalias at aerifal.cx> writes:
> >
> > > On Mon, Dec 31, 2007 at 11:15:39AM +0100, Diego Biurrun wrote:
> > >> > --- configure (revision 11358)
> > >> > +++ configure (working copy)
> > >> > @@ -551,6 +551,23 @@
> > >> >
> > >> > +check_type(){
> > >> > + log check_type "$@"
> > >> > + headers=$1
> > >> > + type=$2
> > >> > + shift 2
> > >> > + disable $type
> > >> > + incs=""
> > >> > + for hdr in $headers; do
> > >> > + incs="$incs
> > >> > +#include <$hdr>"
> > >> > + done
> > >> > + check_cc "$@" <<EOF && enable $type
> > >> > +$incs
> > >> > +$type v;
> > >> > +EOF
> > >> > +}
> > >>
> > >> Ummm, $headers only ever contains one entry, you just assigned $1 to
> > >> it, so what's the point of looping over its content?
> > >
> > > check_type "foo.h bar.h baz.h" faz_t
> >
> > That won't work with the code as-is, since $1 isn't inside quotes.
>
> Word splitting and pathname expansion are not performed on RHS of
> assignments in bourne shell. foo="$bar" is always useless; foo=$bar is
> just as good.
Yes, I was wrong, the patch is good, applied.
Diego
More information about the ffmpeg-devel
mailing list