Vitor wrote: > Hi > > Aurelien Jacobs wrote: > > vitor wrote: > > [...] > > >> } > >> } > >> ++#if ENABLE_AVFILTER > >> + if(ist->picref) > >> + avfilter_unref_pic(ist->picref); > >> -+ > >> ++#endif > > > > IMO, this would be nicer written this way: > > > > if (ENABLE_AVFILTER && ist->picref) > > avfilter_unref_pic(ist->picref); > > > > Aurel > > Would this be valid C99? Because ist has no member named picref if > ENABLE_AVFILTER is false...
Ah... Sorry. I didn't noticed this. So indeed, #if is the right thing to do. Aurel _______________________________________________ FFmpeg-soc mailing list [email protected] http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
