[FFmpeg-devel] [PATCH] socklen_t definition

Dave Yeo daveryeo
Tue Jan 1 20:14:57 CET 2008


On 12/31/07 02:15 am, Diego Biurrun wrote:
> On Sun, Dec 30, 2007 at 05:12:33PM -0800, Dave Yeo wrote:
>> Updated patch attached with unrelated stuff removed.
>> Looking at the Windows headers that come with OpenWatcom socklen_t is 
>> declared in ws2tcpip.h.
> 
> Does this stuff actually work for you?

Yes, tested with default and by adding the typedef to sys/socket.h.

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

What if you have one header that depends on another header? Perhaps a 
function that also needs inttypes.h pulled in as well? Not sure how 
things stand now but at one time quite a few headers depended on other 
headers here with the old EMX libc.

> Diego

Dave




More information about the ffmpeg-devel mailing list