On Tue, Aug 19, 2014 at 01:41:49PM +0000, Carl Eugen Hoyos wrote:
> 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?
> There is an endianess chunk which is used to swap BE <-> LE if necessary (see lavf/mov.c:mov_read_enda()). If you look into the array, I'm changing, there are a bunch of similar occurences around. -- Clément B.
pgpJGuUvBfOP5.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
