[FFmpeg-devel] [PATCH] non-interleaved AVI read packet fix

Maksym Veremeyenko verem
Thu Sep 3 19:02:05 CEST 2009


Michael Niedermayer ???????(??):
> On Thu, Sep 03, 2009 at 06:45:51PM +0300, Maksym Veremeyenko wrote:
[...]
>> so may be it has a sense to simplify this condition too (like in attached 
>> patch) if checking for st->nb_index_entries will be first?
> 
> the patch is ok if tested
i tested it against files caused problem - output has correct number 
video and audio samples.

> but it should be split
> its really a factorization and a bugfix and could be hard to understand
> in a year by someone not familiar with the code if its commited combined

I splitted it into two parts:

*avidec_nb_index_entries_factorized.patch* - st->nb_index_entries check 
factorized

*avidec_non_interleaved_read_packet_fix_v8.patch* - fix a bug with 
reading non-interleaved AVI if one the stream is shorter...

Please apply them if there no other objection....

-- 
________________________________________
Maksym Veremeyenko
-------------- next part --------------
A non-text attachment was scrubbed...
Name: avidec_nb_index_entries_factorized.patch
Type: text/x-patch
Size: 820 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090903/c0744eb3/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: avidec_non_interleaved_read_packet_fix_v8.patch
Type: text/x-patch
Size: 815 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090903/c0744eb3/attachment-0001.bin>



More information about the ffmpeg-devel mailing list