On Wed, Nov 27, 2013 at 02:26:07PM +0100, Vittorio Giovara wrote: > On Tue, Nov 26, 2013 at 8:53 PM, Diego Biurrun <[email protected]> wrote: > > On Tue, Nov 26, 2013 at 11:05:07AM +0100, Vittorio Giovara wrote: > >> --- a/Changelog > >> +++ b/Changelog > >> @@ -50,6 +50,7 @@ version 10: > >> - VP9 decoder > >> - support for decoding through VDPAU in avconv (the -hwaccel option) > >> - codec level stereoscopic metadata handling > >> +- new framepack filter > > > > Why "new"? Was there an old framepack filter? Everything in this > > list could be prefixed with "new". > > Tru dat, but in the very same file you get a few 'new' lines... e.g. > - new avconv options -filter_script and -filter_complex_script, which allow a > filtergraph description to be read from a file > - new interlace filter > - new asetpts filter (same as setpts, but for audio) > - new trim and atrim filters > > so, even though everything in the list could be new, apparently there > is a trend for avfilter features that I'd like to keep following.
It's a completely silly trend. Feel free to eliminate those new monikers. > >> --- /dev/null > >> +++ b/libavfilter/vf_framepack.c > >> @@ -0,0 +1,487 @@ > >> + > >> +static void pack_checkers_frame(FramepackContext *s, AVFrame *dst) > > > > What do you mean by "checkers"? The game? Did you intend to use > > "checkered"? > > Yes, checkers the game, it's the nickname of this format, and follows > the name of the 3D type AV_STEREO3D_CHECKERS. > I can restore to 'checkerboard' function and type names if you prefer. Yes, I think that would be more intuitive and thus descriptive. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
