[FFmpeg-devel] [PATCH] mov.c: fix handling of seek return in read_mfra

Mika Raento mikie at iki.fi
Sat Nov 15 08:02:56 CET 2014


On 15 November 2014 09:00, Mika Raento <mikie at iki.fi> wrote:

> On 14 November 2014 21:51, Michael Niedermayer <michaelni at gmx.at> wrote:
>
>> On Fri, Nov 14, 2014 at 07:12:34PM +0200, Mika Raento wrote:
>> > this would cause mfra to be ignored in files larger than 2G
>> > ---
>> >  libavformat/mov.c | 22 ++++++++++++++--------
>> >  1 file changed, 14 insertions(+), 8 deletions(-)
>>
>> why not just chage the type of ret to int64_t instead of adding a
>> second return variable ?
>>
>
> Done, with a comment on why that works.
>
> Thanks.
>

Ah, I forgot the second reason: with a separate seek_ret I can keep the
return code and not overwrite it with the return value of seeking back. I
prefer the first version of the patch.

    Mika

>
>
>>
>> [...]
>> --
>> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> Dictatorship naturally arises out of democracy, and the most aggravated
>> form of tyranny and slavery out of the most extreme liberty. -- Plato
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>>
>


More information about the ffmpeg-devel mailing list