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

[Carl Eugen Hoyos]

Am Di., 16. Feb. 2021 um 19:28 Uhr schrieb Andreas Rheinhardt
<andreas.rheinhardt at gmail.com>:
>
> 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.

(I thought two would be enough but obviously not.)

No strong opinion here, Carl Eugen