[FFmpeg-devel] [PATCH] socklen_t definition

[Måns Rullgård]

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.

-- 
M?ns Rullg?rd
mans at mansr.com