On Sat, 2008-09-27 at 15:05 -0300, rafael2k wrote: > Hello KH, > OK, it's nice to ifdef things. > I thing it's ok to use > #if LIBAVCODEC_VERSION_INT < ((52<<16)+(0<<8)+0) > for the code before the patch. > > > bye, > rafael diniz > > > Em Saturday 27 September 2008, KH KH escreveu: > > 2008/9/27 rafael2k <[EMAIL PROTECTED]>: > > > Ow, it was quite easy to fix. > > > Can anyone review the patch to fix cinelerra w/ the newer ffmpeg API? > > > I think the fix is correct, as I got it looking at the ffmpeg svn diff. > > > > What if this fix is present while an older ffmpeg api is used ? > > > > In the vlc case (for a sample), the ffmpeg API changes are > > conditionalized... This gives something like this: > > --------------- > > #if LIBAVCODEC_VERSION_INT < ((52<<16)+(0<<8)+0) > > p_sys->p_context->bits_per_sample = > > p_dec->fmt_in.audio.i_bitspersample; #else > > p_sys->p_context->bits_per_coded_sample = > > p_dec->fmt_in.audio.i_bitspersample; > > #endif > > --------------- > > > > Nicolas (kwizart) > >
I just wonder, if that is all that is changed? The discussion on the FFmpeg-devel mailing list (albeit sometime quite strong) resulted in the announcement that r15621 was the last 51 version. Now it seems that there is one (!) change in structure member only?!! What about 'error_resilience' being renamed to 'error_recognition'? Note that libquicktime has been affected too. Regards, Frans. _______________________________________________ Cinelerra mailing list [email protected] https://init.linpro.no/mailman/skolelinux.no/listinfo/cinelerra
