On 3/6/2019 5:07 AM, Carl Eugen Hoyos wrote:
> 2019-03-05 19:26 GMT+01:00, James Almer <jamr...@gmail.com>:
>> On 3/5/2019 3:19 PM, Vittorio Giovara wrote:
>
>>> + if (p->mastering_display) {
>>> + AVMasteringDisplayMetadata *mastering =
>>> av_mastering_display_metadata_create_side_data(frame);
>>> + if (!mastering)
>>> + return AVERROR(ENOMEM);
>>> +
>>> + for (i = 0; i < 3; i++) {
>>
>> for (int i = 0,...)
>
> Being allowed to use doesn't mean it has to be used, no?
> There also is a smaller scope...
>
> Carl Eugen
There's no smaller scope than directly declaring it in the for() line.
But he can add it right below the AVMasteringDisplayMetadata declaration
if that's preferred. If just don't see why adding an extra line to the
file would be preferred in this specific case to begin with.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
