[FFmpeg-devel] [PATCH] mp4 and ipod metadata

[Baptiste Coudurier]

Michael Niedermayer wrote:
> On Tue, Jun 10, 2008 at 10:54:48PM -0700, Baptiste Coudurier wrote:
>> Michael Niedermayer wrote:
>>> On Mon, Jun 09, 2008 at 02:30:34PM -0700, Baptiste Coudurier
>>> wrote:
>>>> Hi all,
>>>> 
>>>> Atm current mp4 muxer writes non standard and itunes specific
>>>> metadata. I intend to change this behaviour and to writes these
>>>> metadata only for -f ipod output format.
>>> Iam against this change unless there is an actual application for
>>> which the tags cause problems (which really shouldnt be,
>>> applications should ignore tags unknown to them, especially if
>>> they are within "user data").
>>> 
>> First, correct and standards tags exists for 3gp, so itunes tags
>> should not be written for 3gp, nor for mov since equivalent exist.
>> 
>> Then, as a general attitude, Im against writing non standard
>> features in perfectly defined standards.
>> 
>> If the user wants them he can use -f ipod format, we are not
>> removing support for metadata. We created a specific format for
>> this, please let's use it and stop spreading that mp4 is
>> ipod/itunes format.
> 
> What i want (and i belive users as well) is a "universal" format
> which can be used everywhere.

Well use nut ;)

> That doesnt mean ipod tags should be in -f mp4 or -f mov but that
> there is a -f foobar that contains all tags of all variants. I prefer
> if i dont need to have each video 3 times so it can be played on 3
> different players.

I understand the need, Im afraid it's complicated.

Unfortunately, it is so brainded to deal with all iso media variants and
their fields having different meaning values that I tend to prefer
constraining specific features to specific formats.
I think that adding this ipod format was a good thing.

Anyway I've added 3gp metadata support.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA