[FFmpeg-devel] [PATCH] flv format support mp3 audio with 48khz

[Michael Niedermayer]

On Wed, Sep 28, 2016 at 05:26:35PM +0200, wm4 wrote:
> On Wed, 28 Sep 2016 07:47:47 -0700
> fuqiuping <fu.qiuping at hotmail.com> wrote:
> 
> > ---
> >  libavformat/flvenc.c |    8 ++++++++
> >  1 files changed, 8 insertions(+), 0 deletions(-)
> > 
> > diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c
> > index 99903f5..296426a 100644
> > --- a/libavformat/flvenc.c
> > +++ b/libavformat/flvenc.c
> > @@ -107,6 +107,13 @@ static int get_audio_flags(AVFormatContext *s, AVCodecParameters *par)
> >          return FLV_CODECID_SPEEX | FLV_SAMPLERATE_11025HZ | FLV_SAMPLESSIZE_16BIT;
> >      } else {
> >          switch (par->sample_rate) {
> > +        case 48000:
> > +            if (par->codec_id == AV_CODEC_ID_MP3) {
> > +                flags |= FLV_SAMPLERATE_44100HZ;
> 
> This looks wrong. It should have a code comment why it's right, even
> though it 'll look wrong to every single person who look at tit for the
> first time.

added comment
applied

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct answer.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161106/93dfd17c/attachment.sig>