[FFmpeg-devel] [PATCH 2/3] Add muxer/demuxer for raw codec2 and .c2 files

Tomas Härdin tjoppen at acc.umu.se
Fri Feb 9 15:43:34 EET 2018


On 2018-02-09 11:29, Carl Eugen Hoyos wrote:
> 2018-01-15 22:36 GMT+01:00 Tomas Härdin <tjoppen at acc.umu.se>:
>
>>> +    if (p->buf[4] >  EXPECTED_CODEC2_MINOR_VERSION) score -=
>>> AVPROBE_SCORE_MAX/5;
>>> +    if (p->buf[5] >  AVPRIV_CODEC2_MODE_MAX)        score -=
>>> AVPROBE_SCORE_MAX/5;
>>> +    if (p->buf[6] != 0)                             score -=
>>> AVPROBE_SCORE_MAX/5;
>>> +    return score;
>>>
>>> Imo, this is too complicated:
>>> If the first 32bit are correct, return MAX/2, for 24bit, return less
> This should have been AVPROBE_SCORE_EXTENSION + 1,
> sorry about my mistake.

Alright. A documentation update may be in order unless already done

> Please threaten to push this and push after a few days.

Don't have ssh access, but I believe Michael has offered to add me 
several times. I'll email him a GPG signed copy of a public SSH key when 
I get home

/Tomas


More information about the ffmpeg-devel mailing list