[FFmpeg-devel] [RFC] LPCM 24 bits support

Giuliano Pochini pochini
Mon Apr 7 22:27:33 CEST 2008


On Mon, 07 Apr 2008 14:56:08 +0200
Baptiste Coudurier <baptiste.coudurier at smartjog.com> wrote:

> Hi,
> 
> Giuliano Pochini wrote:
> > [...]
> > 
> > +    REGISTER_DECODER (PCM_DVD20, pcm_dvd20);
> > +    REGISTER_DECODER (PCM_DVD24, pcm_dvd24);
> 
> IMHO only use one codec id, that is PCM_DVD for > 16 bits, if 16 use
> CODEC_ID_PCM_S16BE.

I think we shouldn't overload a decoder to make it decode two different
formats. One format : one decoder is the cleanest way IMHO and it's how all
other PCM encoders and decoders are arranged. I don't mind to change it if
having them merged is better.


--
Giuliano.




More information about the ffmpeg-devel mailing list