[FFmpeg-devel] [PATCH] avformat/isom: add ni24 typo/alias for in24

Carl Eugen Hoyos cehoyos at ag.or.at
Tue Aug 19 15:41:49 CEST 2014


Clément Bœsch <u <at> pkh.me> writes:

> On Mon, Aug 18, 2014 at 11:39:26PM +0000, Carl Eugen Hoyos wrote:
> > Clément Bœsch <u <at> pkh.me> writes:
> > 
> > > +    { AV_CODEC_ID_PCM_S24BE,       MKTAG('n', 'i', '2', '4') },
> > > BBC typo */
> > > +    { AV_CODEC_ID_PCM_S24LE,       MKTAG('n', 'i', '2', '4') },
> > > /* BBC typo */

> I'm sorry, I don't understand what you mean.

I believe that FFmpeg would not be able to 
decode "ni24" samples with your patch above.
If I understand the patch from Elemental 
correctly, all ni24 files they encountered 
are PCM_S24LE, while your patch would identify 
such files as PCM_S24BE.
Or do I misunderstand?

Carl Eugen



More information about the ffmpeg-devel mailing list