[FFmpeg-devel] [PATCH]Implement max_lowres

Jai Menon jmenon86
Tue Jul 6 06:47:03 CEST 2010


On Tue, Jul 6, 2010 at 3:31 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Hi!
>
> On Monday 05 July 2010 04:40:59 pm Michael Niedermayer wrote:
>
>> > First version attached. Is there an API-change or Changelog entry needed?
>> > Note that this should not be held back because of the obvious
>> > regressions: ffmpeg -lowres is currently broken for default configure
>> > anyway;-)
>
>> i dont think mpeg1/2 are the only that support lowres
>
> As I said, this should currently be no reason to delay this patch.
>
>> [...]> ? ? ?if(avctx->codec->init){
>>
>> > + ? ? ? ?if(avctx->codec->max_lowres < avctx->lowres){
>> > + ? ? ? ? ? ?av_log(avctx, AV_LOG_ERROR, "The maximum value for lowres
>> > supported by the decoder is %d\n", +
>> > avctx->codec->max_lowres);
>> > + ? ? ? ?goto free_and_end;
>> > + ? ? ? ?}
>>
>> indent..
>
> New version attached. libopenjpeg always crashes for me with lowres>0.

Works here till 5 reslevels (beyond which swscale complains)

-- 
Jai Menon



More information about the ffmpeg-devel mailing list