[FFmpeg-devel] [PATCH 10/15] avformat/matroskaenc: Avoid seeking when writing level 1 elements
Andreas Rheinhardt
andreas.rheinhardt at googlemail.com
Tue Apr 2 19:29:00 EEST 2019
Hendrik Leppkes:
> On Tue, Apr 2, 2019 at 3:36 PM Andreas Rheinhardt via ffmpeg-devel
> <ffmpeg-devel at ffmpeg.org> wrote:
>> @@ -383,8 +388,8 @@ static void end_ebml_master_crc32_preliminary(AVIOContext *pb, AVIOContext **dyn
>> uint8_t *buf;
>> int size = avio_get_dyn_buf(*dyn_cp, &buf);
>>
>> + put_ebml_num(pb, size, master.sizebytes);
>> avio_write(pb, buf, size);
>> - end_ebml_master(pb, master);
>> }
>>
>
> The indent here seems off, or did my mail client mangle something?
>
In patch 8 I removed a redundant check in
end_ebml_master_crc32_preliminary, but I did not change indentation
(that's done in patch #12). So put_ebml_num is written at the level of
indentation that the rest will be after the indentation will have been
fixed in #12.
(Alternatively, one could write put_ebml_num one indentation level too
high and shift it to the left in #12.)
- Andreas
More information about the ffmpeg-devel
mailing list