[FFmpeg-devel] [PATCH] avcodec/extract_extradata: zero initialize the padding bytes of the exported extradata

Paul B Mahol onemda at gmail.com
Fri Mar 9 22:53:59 EET 2018


On 3/9/18, wm4 <nfxjfg at googlemail.com> wrote:
> On Fri, 9 Mar 2018 13:31:45 -0300
> James Almer <jamrial at gmail.com> wrote:
>
>> On 3/9/2018 12:47 PM, Derek Buitenhuis wrote:
>> > On 3/9/2018 3:22 PM, James Almer wrote:
>> >> Yes, but it's slower, and the buffer is guaranteed to be written to
>> >> with
>> >> actual data after being allocated.
>> >>
>> >> This is a filter that may run once per processed packet, so the less
>> >> overhead the better.
>> >
>> > Not sure I buy the "speed" argument here, but OK.
>> >
>> > - Derek
>>
>> There's really nothing to win zeroing then immediately writing actual
>> data on top of it.
>>
>
> Except code clarity and robustness?

Code already is clear and robust enough.


More information about the ffmpeg-devel mailing list