[FFmpeg-cvslog] r11815 - trunk/libavformat/asf.c

Rich Felker dalias
Sat Feb 2 22:46:27 CET 2008


On Sat, Feb 02, 2008 at 10:25:35PM +0100, Michael Niedermayer wrote:
> On Sat, Feb 02, 2008 at 10:17:46PM +0100, reimar wrote:
> > Author: reimar
> > Date: Sat Feb  2 22:17:46 2008
> > New Revision: 11815
> > 
> > Log:
> > Put is_mms under ifdef CONFIG_MMSH_PROTOCOL, avoids warning:
> > libavformat/asf.c:112: warning: 'is_mms' defined but not used
> 
> Iam not too much in favor of these kind of warning fixes ...

Agree. cost(#ifdef) >> cost(warning) (and here >> means greater by an
order of magnitude, not rightshift).

Rich




More information about the ffmpeg-cvslog mailing list