[FFmpeg-devel] [PATCH] add uuid box to MP4 to make h.264 play on iPod

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> On Sat, Feb 23, 2008 at 12:25:16PM +0000, M?ns Rullg?rd wrote:
>> Mark Himsley <mark at mdsh.com> writes:
>> 
>> > Attached patch adds new -f <format> of ipod264. When using that format
>> > the file is created as with mp4 format but with the addition of the
>> > uuid required for the file to be transferred from iTunes to an iPod
>> > that plays "Low Complexity" Baseline Profile H.264. Tested syncing
>> > with iPod Touch.
>> 
>> I don't like inventing new, fake formats for each and every
>> variation.  
>
> I dont like it either but its easy, simple and doesnt really add hard to
> maintain code ...
> Also changing the fake formats to some other system is seperate of adding
> new ones IMHO.

The more new ones are add, the more work there will be if we want to
switch to something better.

>> Things like this should be passed as flags/parameters to
>> the mp4 muxer instead.  If there is no way of doing that currently,
>> lavf should be extended to support it.
>
> This causes problems, because the following is stored per AVOutputFormat:
> * default codecs
> * list of supported codecs as well as their codec tags

This could be solved by moving those lists into an AVSubFormat (or
whatever) struct, along with a name and whatever else might be useful.
Each AVOutputFormat would then have a list of AVSubFormat (the first
being the default).

Yes, this would break compatibility, and no, I haven't though about it
in depth.

-- 
M?ns Rullg?rd
mans at mansr.com