On Sun, 01 May 2016 00:09:46 +0200
Piotr Bandurski <ami_st...@o2.pl> wrote:

> > > On Wed, 27 Apr 2016 20:39:34 +0200
> > Piotr Bandurski <ami_st...@o2.pl> wrote:
> > 
> > > Subject: [PATCH] avformat/riff: assign g721 and g723 codec tags to
> > > g726 decoder
> > 
> > > + { AV_CODEC_ID_ADPCM_G726, 0x0014 },
> > > + { AV_CODEC_ID_ADPCM_G726, 0x0040 },
> > 
> > i wonder if we should make a comment that these are different
> > codecs, in the unlikely eventuality that we split 721 723 and 726
> > decoders from one another?
> > 
> > > + { AV_CODEC_ID_ADPCM_G726, 0x0014 }, /* g723 Antex */
> > > + { AV_CODEC_ID_ADPCM_G726, 0x0040 }, /* g721 Antex */
> > 
> > ?
> 
> I'm ok with this change.

merged,

thanks
-compn
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to