[FFmpeg-devel] [PATCH 04/10] avcodec/dca: don't set initial sample_fmt

[Hendrik Leppkes]

On Fri, May 13, 2016 at 11:48 AM, foo86 <foobaz86 at gmail.com> wrote:
> Valid sample_fmt will be set by dcadec_decode_frame() based on stream
> type.
> ---
>  libavcodec/dcadec.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c
> index e3a4b0d..565242d 100644
> --- a/libavcodec/dcadec.c
> +++ b/libavcodec/dcadec.c
> @@ -374,9 +374,6 @@ static av_cold int dcadec_init(AVCodecContext *avctx)
>          break;
>      }
>
> -    avctx->sample_fmt = AV_SAMPLE_FMT_S32P;
> -    avctx->bits_per_raw_sample = 24;
> -
>      return 0;
>  }
>

Having an initial format for the most "common" types of streams is
generally sensible.

- Hendrik