[FFmpeg-cvslog] r9623 - trunk/libavformat/rtp_h264.c

[Alex Beregszaszi]

Hi,

> >> Modified: trunk/libavformat/rtp_h264.c
> >> ==============================================================================
> >> --- trunk/libavformat/rtp_h264.c	(original)
> >> +++ trunk/libavformat/rtp_h264.c	Fri Jul 13 14:12:38 2007
> >> @@ -165,9 +165,7 @@ static int h264_handle_packet(RTPDemuxCo
> >>                                const uint8_t * buf,
> >>                                int len)
> >>  {
> >> -#ifdef DEBUG
> >>      h264_rtp_extra_data *data = s->dynamic_protocol_context;
> >> -#endif
> >>     
> >
> > Maybe adding a av_unused would be better ? (I didn't checked if
> > it generate a warning)
> >
> >   
> 
> FWIW, it does generate a warning...

My bad, however the assert() are compiled anyway, so this breaks. Maybe
that ifdef should be #if defined(NDEBUG) || defined(DEBUG) ?

Or leave it there with av_unused for the time being.

--
Alex