[FFmpeg-devel] [PATCH] provide an input parameter to specify 3d stereo format for mkv/webm 3d videos

Alok Ahuja waveletcoeff at gmail.com
Tue Apr 26 08:47:17 CEST 2011

Thanks for the feedback. I've updated the patch with recommendations.

Some comments
* This patch only addresses the StereoMode tag which is shared by both mkv
and webm. We can look at enabling TrackOperation in a subsequent patch.
* I'm not sure of an inherent need to provide demux support for the same tag
since it is up to a specific implementation as to how they would operate on
the StereoMode tag. For example, Firefox uses it to render 3D WebM videos.
* Have multiple video tracks containing StereoMode tags within a single
container does not seem like a valid use-case. Therefore, it should be ok to
limit the input to a single stereo_mode parameter.


On Sun, Apr 24, 2011 at 2:39 AM, Reimar Döffinger
<Reimar.Doeffinger at gmx.de>wrote:

> On Sun, Apr 24, 2011 at 12:56:37PM +0400, Kirill Gavrilov wrote:
> >    - this flag is written per video track, but your patch writes the
> *common
> >    * flag for ALL video tracks; this is wrong, but if it is simplest way
> to
> >    save this flag using existing API...;
> I don't think so, if you just use metadata to pass it, it should be rather
> easy.
> It also would keep the information when converting from/to formats that
> only
> support generic metadata but not this specific one.
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Create-a-stereo_mode-metadata-tag-to-specify-the-ste.patch
Type: application/octet-stream
Size: 3590 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110425/5e0353e5/attachment.obj>

More information about the ffmpeg-devel mailing list