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

[Maksym Veremeyenko]

Benjamin Larsson ???????(??):
> Maksym Veremeyenko wrote:
>> Diego Biurrun ???????(??):
>>> On Thu, Sep 03, 2009 at 08:27:07AM +0300, Maksym Veremeyenko wrote:
>>>> Newer patch attached.
>>>>
>>>> --- libavformat/avidec.c    (revision 19698)
>>>> +++ libavformat/avidec.c    (working copy)
>>>> @@ -680,6 +680,12 @@
>>>>  
>>>> +            /* skip finished stream */
>>>> +            if(!ast->remaining && st->nb_index_entries &&
>>>> +               ts > st->index_entries[st->nb_index_entries - 
>>>> 1].timestamp *
>>>> +                    FFMAX(1, ast->sample_size))
>>>
>>> Indentation is off.
>> [...]
>> Sorry, i do not see...
>> Which line?
>>
>>
> ts > st->index_entries[st->nb_index_entries - 1].timestamp *
> 
> is only 3 spaces indented.
if i correctly understand it should be aligned with *ast* variable name 
instead of ! sign? Is that proper:

if(!ast->remaining && st->nb_index_entries &&
     ts > st->index_entries[st->nb_index_entries - 1].timestamp *
          FFMAX(1, ast->sample_size))

newer patch attached

-- 
________________________________________
Maksym Veremeyenko

-------------- next part --------------
A non-text attachment was scrubbed...
Name: avidec_non_interleaved_read_packet_fix_v5.patch
Type: text/x-patch
Size: 688 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090903/8d772491/attachment.bin>