[FFmpeg-devel] [PATCH 1/2] clarified the usage of ISO 639 language codes

[Stefano Sabatini]

On date Wednesday 2014-01-08 12:48:48 +0100, Michael Niedermayer encoded:
> On Wed, Jan 08, 2014 at 10:02:15AM +0100, Stefano Sabatini wrote:
> > On date Wednesday 2014-01-08 03:57:01 -0500, Helmut Tessarek encoded:
> > > On 08.01.14 3:33 , Stefano Sabatini wrote:
> > > > [avs]lang options were removed in
> > > > 7db7eb08e051d0d95215f645e0cfa9a5879cded0, the corresponding option
> > > > documentation should be removed.
> > > 
> > > No argument here.
> > > 
> > > I just thought since they are still in the docs I update these sections as well.
> > > 
> > > What's the next step? Shall I remove them or is anybody else already doing this?
> > 
> > I deleted the remainings of the removed options.
> > 
> 
> > About the example comment, I'm not sure it's a good idea to add since
> > the "language" metadata is interpreted differently from each
> > container, and thus the use of ISO codes could be unrelevant.
> 
> if you suggest that the user has to read and understand the
> specification of a format and implemententation before he can set the
> language in that container. Than frankly i think you can remove
> support for setting languages completely.

I never implied that.

> 
> The code must remap the language codes to what the destination
> supports, otherwise it just cant be used, not even copying the
> language code from one container to the other could work

Still not all containers honour metadata, and they might differ how
they interpret the language metadata value. Anyway feel free to add
the example comment or suggest whatever you think is missing in the
docs.
-- 
FFmpeg = Fancy Fascinating Magic Pure Enlightened Game