On Mon, Aug 24, 2015 at 11:08 PM, wm4 <nfx...@googlemail.com> wrote:
> On Mon, 24 Aug 2015 20:18:59 +0200
> Michael Niedermayer <michae...@gmx.at> wrote:
>
>> From: Michael Niedermayer <mich...@niedermayer.cc>
>>
>> See Ticket 4770
>>
>> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
>> ---
>> libavcodec/mjpegenc.c | 9 ---------
>> 1 file changed, 9 deletions(-)
>>
>> diff --git a/libavcodec/mjpegenc.c b/libavcodec/mjpegenc.c
>> index ee0b16e..ae0c8cb 100644
>> --- a/libavcodec/mjpegenc.c
>> +++ b/libavcodec/mjpegenc.c
>> @@ -228,15 +228,6 @@ static int amv_encode_picture(AVCodecContext *avctx,
>> AVPacket *pkt,
>> if(s->avctx->flags & CODEC_FLAG_EMU_EDGE)
>> return AVERROR(EINVAL);
>>
>> - if ((avctx->height & 15) && avctx->strict_std_compliance >
>> FF_COMPLIANCE_UNOFFICIAL) {
>> - av_log(avctx, AV_LOG_ERROR,
>> - "Heights which are not a multiple of 16 might fail with some
>> decoders, "
>> - "use vstrict=-1 / -strict -1 to use %d anyway.\n",
>> avctx->height);
>> - av_log(avctx, AV_LOG_WARNING, "If you have a device that plays AMV
>> videos, please test if videos "
>> - "with such heights work with it and report your findings to
>> ffmpeg-devel@ffmpeg.org\n");
>> - return AVERROR_EXPERIMENTAL;
>> - }
>> -
>> pic = av_frame_clone(pic_arg);
>> if (!pic)
>> return AVERROR(ENOMEM);
>
> I don't quite understand why decoding an AMV file (whatever that is)
> requires a change in the mjpeg encoder?
It doesn't, but presumably the existence of other non-mod16 AMV files
has prompted the question if its maybe not that bad to create them.
- Hendrik
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
