Re: [PATCH] alsa-lib: Add sys/types.h to include list

[David Henningsson <david.henningsson@canonical.com>]

On 02/12/2013 10:10 AM, Takashi Iwai wrote:
> At Tue, 12 Feb 2013 10:06:11 +0100,
> David Henningsson wrote:
>>
>> This is needed by snd_pcm_format_silence* functions which
>> return u_int*_t. It was discovered while trying to compile ALSA
>> programs with eglibc 2.17.
>>
>> Credits to Richard Shaw, Gary Buhrmaster, Matthieu Baerts and
>>   Adam Conrad for this fix.
>>
>> BugLink: https://bugs.launchpad.net/bugs/1109298
>> BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=885306
>> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
>
> For u_int_* types, wouldn't it better to include stdint.h?

Hmm, it looks like stdint.h declares uint_* whereas sys/types.h declares 
u_int_* (notice the _ between "u" and "int"). Do you think we should 
change snd_pcm_format_silence* from u_int to uint?

>
>
> Takashi
>
>> ---
>>   include/asoundlib-head.h |    1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/include/asoundlib-head.h b/include/asoundlib-head.h
>> index 20c8a68..6edbab0 100644
>> --- a/include/asoundlib-head.h
>> +++ b/include/asoundlib-head.h
>> @@ -31,6 +31,7 @@
>>   #include <unistd.h>
>>   #include <stdio.h>
>>   #include <stdlib.h>
>> +#include <sys/types.h>
>>   #include <string.h>
>>   #include <fcntl.h>
>>   #include <assert.h>
>> --
>> 1.7.9.5
>>
>



-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic