Re: [PATCH] tools.h: include `stdarg.h` to fix `error: 'va_list' has not been declared` with uClibc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12.12.2010 23:03, Paul Menzel wrote:
> From a94d15b582df5d1464cec8bc7dd68be46e1bf937 Mon Sep 17 00:00:00 2001
> From: Paul Menzel <paulepanter@xxxxxxxxxxxxxxxxxxxxx>
> Date: Sun, 12 Dec 2010 19:21:25 +0100
> Subject: [PATCH] tools.h: include `stdarg.h`
> 
> For some reason using uClibc `tools.h` needs to have `stdarg.h` included explicitly. Otherwise it fails with the following error. Using Libc or EGLIBC this error does not surface.

Since tools.h uses va_list you're apparently right.
I wonder which one of the other header files tools.h
includes pulls in that declaration (apparently unnecessarily).

> ...
> This change was made by Henning Heinhold when packaging VDR 1.7.10 for OpenEmbedded [1].
> 
> [1] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/vdr/files/cplusplus.patch

So I'll mention Henning in the CONTRIBUTORS file then, if that's ok with you.

> Signed-off-by: Paul Menzel <paulepanter@xxxxxxxxxxxxxxxxxxxxx>
> CC: Henning Heinold <heinold@xxxxxxxxxxxxxxxx>
> ---
>  tools.h |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/tools.h b/tools.h
> index 95c35ff..1fbe005 100644
> --- a/tools.h
> +++ b/tools.h
> @@ -17,6 +17,7 @@
>  #include <iconv.h>
>  #include <math.h>
>  #include <poll.h>
> +#include <stdarg.h>
>  #include <stddef.h>
>  #include <stdint.h>
>  #include <stdio.h>

The '#include <stdarg.h>' can then also be removed from tools.c,
and apparently it's included unnecessarily in osd.c and receiver.c.

Klaus

_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux