On Wed, 2 Oct 2013 10:21:33 +0400, Kirill Gavrilov <[email protected]> wrote: > On Wed, Oct 2, 2013 at 9:35 AM, Anton Khirnov <[email protected]> wrote: > > > > + > > > +/* How views are packed within the frame or container*/ > > > +enum AVStereo3DType { > > > + /** > > > + * Video is not stereoscopic > > > + */ > > > + AV_STEREO3D_NONE, > > > + > > > + /** > > > + * Video is not stereoscopic but metadata has to be there > > > + */ > > > + AV_STEREO3D_NOT_REALLY, > > > > Can't say I like the name. Does this thing even have to exist? Can't we use > > AV_STEREO3D_NONE for this? > > If I understand context correctly, AV_STEREO3D_NONE should be something > like AV_STEREO3D_UNDEFINED/AV_STEREO3D_UNKNOWN (which means that file/frame > doesn't contain stereoscopic information at all, but apparently might > contain stereoscopic content in unknown format), and AV_STEREO3D_NOT_REALLY > should be just AV_STEREO3D_NONE (which means the file contains stereoscopic > tags).
I don't quite see why would we need to explicitly signal that the file does not contain 3d metadata. We can just not set the frame metadata to the same effect. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
