[FFmpeg-devel] [PATCH] Clip-Wrapped MXF support (attempt #4)

Maksym Veremeyenko verem
Wed Jan 26 10:11:29 CET 2011


Reimar D?ffinger ???????(??):
> On Tue, Jan 25, 2011 at 01:38:10PM +0200, Maksym Veremeyenko wrote:
>> @@ -524,13 +530,35 @@ static int mxf_read_source_package(void *arg, ByteIOContext *pb, int tag, int si
>>  
>>  static int mxf_read_index_table_segment(void *arg, ByteIOContext *pb, int tag, int size, UID uid)
>>  {
>> +    MXFIndexTableSegment *segment = (MXFIndexTableSegment *)arg;
> 
> Pointless cast.

will fix

> 
>> +                /* check small EditUnitByteCount for audio */
>> +                if (!mxf->current_klv_bsize)
>> +                    mxf->current_klv_bsize = 1024;
> 
> IMO this "default" is a lot too small, one packet per 1 kB is a
> massive overhead if you end up using it for e.g. a 100 MBit/s
> MPEG-2 just because it does not have any such metadata setting.
> 1 kB sized packets means > 2% memory overhead in general and
> for above example it would be about 12000 demuxer calls per second!
> Apart from that I retract my earlier objections, I wasn't aware
> the whole file would be read into memory currently.

does 1440000 bytes block is good enough - that magic count odd to 
DV25...50 frames...


-- 
________________________________________
Maksym Veremeyenko



More information about the ffmpeg-devel mailing list