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

[Reimar Döffinger]

On Mon, Apr 25, 2011 at 11:47:17PM -0700, Alok Ahuja wrote:
> Thanks for the feedback. I've updated the patch with recommendations.

I think it would be more user-friendly to allow an actual string instead
of just numbers, also specifying a value that is not supported should at
least print a warning or even fail.

> * 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.

If you don't simple remuxing (e.g. to convert mkv to WebM or to add an
extra audio track) will remove the 3D info.

> * 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.

Why shouldn't it be valid to want to store both anaglyph and "proper" 3D
in a single file?
It seems really silly to not support that when the only change is to use
st->metadata instead of s->metadata, not supporting a feature to save
one character of code?!