[FFmpeg-devel] qt-faststart uuid take 2

[Frank Barchard]

This patch allows qt-faststart to work on MOV files with UUID top level atoms.

Before the change, the tool would fail:

qt-faststart faces.mp4 bob.mov
encountered non-QT top-level atom (is this a Quicktime file?)
last atom in file was not a moov atom

qt-faststart candles.mp4 bob.mov
encountered non-QT top-level atom (is this a Quicktime file?)
last atom in file was not a moov atom

With the fix, the files are parsed correctly:

qt-faststart faces.mp4 bob.mov
ftyp          0 28
uuid         28 148
mdat        176 43981826
moov   43982002 22107
 patching stco atom...
 patching stco atom...
 writing ftyp atom...
 writing moov atom...
 copying rest of file...

qt-faststart candles.mp4 bob.mov
ftyp          0 24
moov         24 58817
mdat      58841 35960979
uuid   36019820 14670
last atom in file was not a moov atom

The atom printf was used to track down the new atom and improves the
error reporting.
The offset is computed, rather than use ftello, for better portability.
Caveat: The issue with candle.mp4 is fixable (move mdat last), but
substantially more code, so I've deferred that.
Patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: qt-faststart-uuid.diff
Type: application/octet-stream
Size: 3422 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090702/fece766e/attachment.obj>