On 12.12.2010 23:03, Paul Menzel wrote:
> From a94d15b582df5d1464cec8bc7dd68be46e1bf937 Mon Sep 17 00:00:00 2001
> From: Paul Menzel <paulepan...@users.sourceforge.net>
> 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 <paulepan...@users.sourceforge.net>
> CC: Henning Heinold <hein...@inf.fu-berlin.de>
> ---
>  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@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr

Reply via email to