2018-03-21 21:32 GMT+01:00, Carl Eugen Hoyos <ceffm...@gmail.com>:
> 2018-03-21 14:37 GMT+01:00, Gyan Doshi <gyando...@gmail.com>:
>> Fix for ticket 7091.
>
>> -            film->sample_table[i].keyframe = (scratch[8] & 0x80) ? 0 :
>> 1;
>> +            film->sample_table[i].keyframe = (scratch[8] & 0x80) ? 1 :
>> 0;
>
> Can you comment on why this 15-year old code was wrong?
>
> Could be keyframe = !!(scratch & 0x80)

Or actually:
film->sample_table[i].keyframe = scratch[8] & 0x80 ? AVINDEX_KEYFRAME : 0;

Carl Eugen
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to