[FFmpeg-devel] [PATCH]lavc/pnm_parser: Add a pfm parser

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Tue Feb 16 20:28:37 EET 2021


Carl Eugen Hoyos:
> Am Di., 16. Feb. 2021 um 00:56 Uhr schrieb Andreas Rheinhardt
> <andreas.rheinhardt at gmail.com>:
>>
>> Carl Eugen Hoyos:
>>> Hi!
>>>
>>> Attached patch is necessary for a new pfm demuxer.
>>>
>>> Please comment, Carl Eugen
>>>
>> Can't we just wait and increment the number of codec_ids
>> a parser can support at the next bump?
> 
> I wanted to suggest to reduce the number because for
> 90% of the parsers, one is enough and in one case,
> six may not be enough in the future...
> 
Parsers with multiple codec-ids:
ac3: AC3, EAC3
g729: G729, ACELP_KELVIN
gsm: GSM, GSM_MS
mjpeg: MJPEG, JPEGLS
mlp: MLP, TRUEHD
mpegaudio: MP1, MP2, MP3, MP3ADU
mpegvideo: MPEG1VIDEO, MPEG2VIDEO
pnm: PGM, PGMYUV, PPM, PBM, PAM
vp3: THEORA, VP3, VP6, VP6F, VP6A

We have 52 AVCodecParsers in total (for 69 codec ids) of which 9 use
multiple codec ids; the parsers with multiple codec ids have 26 codec
ids together. Either way that is way more than 10% and is not niche.
And I actually thought about incrementing it to seven.

- Andreas


More information about the ffmpeg-devel mailing list