Hi,
On Mon, Jun 19, 2017 at 2:05 PM, Ronald S. Bultje <rsbul...@gmail.com>
wrote:
> Hi,
>
> On Mon, Jun 19, 2017 at 11:10 AM, James Darnley <jdarn...@obe.tv> wrote:
>
>> ---
>> libavcodec/mdec.c | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/mdec.c b/libavcodec/mdec.c
>> index 8e28aa04f0..97bfebbeb7 100644
>> --- a/libavcodec/mdec.c
>> +++ b/libavcodec/mdec.c
>> @@ -213,6 +213,9 @@ static av_cold int decode_init(AVCodecContext *avctx)
>> {
>> MDECContext * const a = avctx->priv_data;
>>
>> + if (avctx->idct_algo == FF_IDCT_AUTO)
>> + avctx->idct_algo = FF_IDCT_SIMPLE;
>> +
>> a->mb_width = (avctx->coded_width + 15) / 16;
>> a->mb_height = (avctx->coded_height + 15) / 16;
>>
>> @@ -225,8 +228,6 @@ static av_cold int decode_init(AVCodecContext *avctx)
>> ff_init_scantable(a->idsp.idct_permutation, &a->scantable,
>> ff_zigzag_direct);
>>
>> - if (avctx->idct_algo == FF_IDCT_AUTO)
>> - avctx->idct_algo = FF_IDCT_SIMPLE;
>> avctx->pix_fmt = AV_PIX_FMT_YUVJ420P;
>> avctx->color_range = AVCOL_RANGE_JPEG;
>>
>> --
>> 2.13.1
>
>
> Can we use per-codec defaults to accomplish this?
>
I'm not actually sure this is a good idea, because it changes the meaning
of _AUTO slightly. Patch is OK as-is.
Ronald
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
