[FFmpeg-devel] [PATCH 2/3] mjpegenc: disable huffman coding with AMV

[Rostislav Pehlivanov]

On 4 April 2017 at 00:08, Rostislav Pehlivanov <atomnuker at gmail.com> wrote:

>
>
> On 4 April 2017 at 00:02, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
>
>> 2017-04-03 22:13 GMT+02:00 Rostislav Pehlivanov <atomnuker at gmail.com>:
>>
>> > +    if (s->huffman && avctx->codec_id == AV_CODEC_ID_AMV) {
>> > +        av_log(avctx, AV_LOG_WARNING, "AMV doesn't support optimized
>> huffman tables, disabling\n");
>>
>> Iiuc, together with 3/3 this prints a warning by default when encoding
>> AMV, this should really be avoided.
>>
>> Carl Eugen
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
> I didn't think about that. Changed locally to never warn and instead
> always turn it off with AMV encoding.
>
>
Pushed to master and release/3.3 with the suggested change, thanks!