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

wm4 nfxjfg at googlemail.com
Wed Sep 28 18:26:35 EEST 2016


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.

> +                break;
> +            } else {
> +                goto error;
> +            }
>          case 44100:
>              flags |= FLV_SAMPLERATE_44100HZ;
>              break;
> @@ -124,6 +131,7 @@ static int get_audio_flags(AVFormatContext *s, AVCodecParameters *par)
>                  break;
>              }
>          default:
> +error:
>              av_log(s, AV_LOG_ERROR,
>                     "FLV does not support sample rate %d, "
>                     "choose from (44100, 22050, 11025)\n", par->sample_rate);



More information about the ffmpeg-devel mailing list