On Sat, 5 Oct 2013 21:14:40 +0200, Vittorio Giovara <[email protected]> wrote: > This structure hosts optional information about > frame composition, frame packing and frame state. > --- > > I reworked the stereo3d patch into something much more general. Now there is a > type which determines what the surface contains, and if not unknown or 2d how > many views are present and how they are relative to each other. Also I've > allocated some space for John's 'showall' patch so that avframe abi can stay > the same. If the type structure is ok for everyone I'll implement the > alloc/free functions and backport my h264/mpeg2 frame packing patches. >
Sorry, I don't think this is the right direction. IMO it tries to overgeneralise for complicated use cases that do not actually exist. It also clumps mostly unrelated things together. I'd prefer it if you went back to the original less ambitious scheme. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
