On 9/20/2017 3:18 PM, wm4 wrote:
> On Wed, 20 Sep 2017 04:00:28 +0100
> Rostislav Pehlivanov <atomnu...@gmail.com> wrote:
>
>> + if (mdm.has_gamma || mdm.has_primaries) {
>> + AVMasteringDisplayMetadata *new_mdm =
>> av_mastering_display_metadata_create_side_data(p);
>> + memcpy(new_mdm, &mdm, sizeof(AVMasteringDisplayMetadata));
>> + }
>> +
>
> Use assignment instead of memcpy, duh.
He shouldn't be using AVMasteringDisplayMetadata on stack to begin with,
so no.
It's bad enough sizeof() has to be used at all already.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
