[FFmpeg-devel] [PATCH][RFC] add a list of generic tag names

[Anton Khirnov]

On Wed, Oct 21, 2009 at 12:35:36AM +0200, Aurelien Jacobs wrote:
> On Tue, Oct 20, 2009 at 03:31:23PM +0200, Anton Khirnov wrote:
> 
> Indeed, this one is quite self-explanatory.
> Not sure if this is true but maybe you can specify something about the
> format of the content, like:
>   you may prefer using id3v1 predefined genre name for best compatibility
> 
I'd prefer if ID3v1-style genres finally disappeared, so I don't really like
the idea of encouraging their use in ffmpeg.

> 
> "name" is not a standard matroska tag, thus it has nothing to do
> in this table.
> 
> Just use "title" directly.
> 
Matroska specs say, that element 536E is called "name". But it's your
file, so changed to "title".

>                                                      ^^^^^
> alignment
> 
fixed
> 
> unrelated cosmetic
> 
fixed

Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-a-list-of-generic-tags.patch
Type: text/x-diff
Size: 9958 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091021/b573572d/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-apetag-Add-metadata-conversion-table.patch
Type: text/x-diff
Size: 2017 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091021/b573572d/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091021/b573572d/attachment.pgp>