[FFmpeg-devel] [PATCH] ffserver: put gcc attribute under proper ifdef

Janne Grunau janne-ffmpeg
Mon Jan 24 23:03:52 CET 2011


On Sun, Jan 23, 2011 at 06:49:32PM +0000, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Sun, Jan 23, 2011 at 06:31:31PM +0000, Mans Rullgard wrote:
> >> ---
> >>  ffserver.c |    5 ++++-
> >>  1 files changed, 4 insertions(+), 1 deletions(-)
> >> 
> >> --- a/ffserver.c
> >> +++ b/ffserver.c
> >> @@ -384,7 +384,10 @@ static void http_vlog(const char *fmt, va_list vargs)
> >>  
> >> -static void __attribute__ ((format (printf, 1, 2))) http_log(const char *fmt, ...)
> >> +#ifdef __GNUC__
> >> +__attribute__ ((format (printf, 1, 2)))
> >> +#endif
> >> +static void http_log(const char *fmt, ...)
> >
> > Is there no way to avoid this non-portable attribute?  Otherwise OK.
> 
> Removing it entirely is of course an option, but then we'd lose the
> format/argument checking enabled by it.

since we have it already, it should stay. ok to commit

Janne



More information about the ffmpeg-devel mailing list