Hi,

On Fri, Jul 1, 2011 at 1:57 PM, Alex Converse <[email protected]> wrote:
> On Fri, Jul 1, 2011 at 11:02 AM, Ronald S. Bultje <[email protected]> wrote:
>> Hi,
>>
>> On Wed, Jun 29, 2011 at 1:30 PM, Alex Converse <[email protected]> 
>> wrote:
>>> -        if(i==0 && pos > avi->movi_list)
>>> -            avi->movi_list= 0; //FIXME better check
>>> -        pos += avi->movi_list;
>>
>> Can movi_list now be replaced by data_offset and then removed from the struct
>>
>
> We now seek to movi_list to compute data offset.
>
>    idx1_pos = avio_tell(pb);
>    avio_seek(pb, avi->movi_list+4, SEEK_SET);
>    if (avi_sync(s, 1) == 0) {
>        first_packet_pos = avio_tell(pb) - 8;
>    }
>    avi->stream_index = -1;
>    avio_seek(pb, idx1_pos, SEEK_SET);

Oh right, I missed that. Well, then I guess the patch can be committed.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to