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

Benoit Fouet benoit.fouet
Fri Jul 13 15:07:19 CEST 2007


Alex Beregszaszi wrote:
> 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.
>
>   

maybe the av_unused is enough

-- 
Ben
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-cvslog mailing list