Hi, On Mittwoch, 12. August 2009, Benoit Fouet wrote: > Hi, > > faust3 wrote : > > Modified: wmapro/wmaprodec.c > > ========================================================================= > >===== --- wmapro/wmaprodec.c Wed Aug 12 20:51:19 2009 (r5042) > > +++ wmapro/wmaprodec.c Wed Aug 12 20:55:40 2009 (r5043) > > @@ -276,12 +276,10 @@ static av_cold int decode_init(AVCodecCo > > s->decode_flags = AV_RL16(edata_ptr+14); > > channel_mask = AV_RL32(edata_ptr+2); > > s->bits_per_sample = AV_RL16(edata_ptr); > > -#ifdef DEBUG > > /** dump the extradata */ > > for (i=0 ; i<avctx->extradata_size ; i++) > > - av_log(avctx, AV_LOG_DEBUG, "[%x] ",avctx->extradata[i]); > > - av_log(avctx, AV_LOG_DEBUG, "\n"); > > -#endif > > + dprintf(avctx, "[%x] ",avctx->extradata[i]); > > + dprintf(avctx, "\n"); > > instead of generating no code at all, this will be a loop of nop, is it > really what you want ? >
It should not really matter as it is called only once during init. Besides that this code will now also work when dprintf no longer depends on DEBUG. Regards Sascha _______________________________________________ FFmpeg-soc mailing list [email protected] https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
