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 <email address hidden>
>
> 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?
On 02/12/2013 10:10 AM, Takashi Iwai wrote: format_ silence* functions which /bugs.launchpad .net/bugs/ 1109298 /bugzilla. redhat. com/show_ bug.cgi? id=885306
> At Tue, 12 Feb 2013 10:06:11 +0100,
> David Henningsson wrote:
>>
>> This is needed by snd_pcm_
>> 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:/
>> BugLink: https:/
>> Signed-off-by: David Henningsson <email address hidden>
>
> 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 format_ silence* from u_int to uint?
u_int_* (notice the _ between "u" and "int"). Do you think we should
change snd_pcm_
> asoundlib- head.h | 1 + asoundlib- head.h b/include/ asoundlib- head.h asoundlib- head.h asoundlib- head.h
>
> Takashi
>
>> ---
>> include/
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/include/
>> index 20c8a68..6edbab0 100644
>> --- a/include/
>> +++ b/include/
>> @@ -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
>>
>
-- /launchpad. net/~diwic
David Henningsson, Canonical Ltd.
https:/