[FFmpeg-devel] [PATCH 6/6] libavformat/movenc: add dnxhr compatibility for apple players

Mark Reid mindmark at gmail.com
Wed Jul 6 03:27:25 EEST 2016


On Tue, Jul 5, 2016 at 1:53 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Mark Reid <mindmark <at> gmail.com> writes:
>
>>      if (track->vos_data && track->vos_len > 0x29) {
>> -        if (track->vos_data[0] == 0x00 &&
>> -            track->vos_data[1] == 0x00 &&
>> -            track->vos_data[2] == 0x02 &&
>> -            track->vos_data[3] == 0x80 &&
>> -            (track->vos_data[4] == 0x01 || track->vos_data[4] == 0x02)) {
>> +        if (avpriv_dnxhd_parse_header_prefix(track->vos_data) != 0) {
>
> This hunk and the two following hunks look unrelated
> to me.
> They could be ok independently of the other change.
>

sure I'll break those changes into a separate patch.


More information about the ffmpeg-devel mailing list