[FFmpeg-devel] [PATCH] mxfdec: Parse IndexTableSegments and convert them into AVIndexEntry arrays

[Georg Lippitsch]

Am 06.10.2011, 17:20 Uhr, schrieb Tomas Härdin <tomas.hardin at codemill.se>:

> I've split this patch in two parts per Baptiste's request.
> The first part reads and converts the IndexTableSegments into
> AVIndexEntry arrays while the second part implements the clip wrapping.

 From my perspective, your patches look fine!

> * handling audio duration % 8192 != 0

Actually I didn't like this kind of audio handling very much, so I wrote  
another patch for this.
After all, it avoids parsing the index table and furthermore generating  
st->index_entries if there is only one table-segment, and if this segments  
has no entries but only EditUnitByteCount set. It is meant to handle the  
audio-files with small EditUnitByteCount correctly, and certainly also  
saves much memory in this case.

Besides that, I have no further comments on the patches for now.

Regards,

Georg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-mxfdec-Avoid-index-table-parsing-if-only-a-single-se.patch
Type: text/x-patch
Size: 4017 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111008/692f3c9f/attachment.bin>