[FFmpeg-devel] [PATCH] allow setting png compression level

Benjamin Larsson banan
Sun Oct 7 11:15:54 CEST 2007


Reimar D?ffinger wrote:
> Hello,
> On Sat, Oct 06, 2007 at 11:42:30PM +0100, M?ns Rullg?rd wrote:
>> Reimar D?ffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> writes:
>>> attached patch would allow setting png compression level via the
>>> coder_type option. This might be considered a bit of a misuse, but I
>>> personally find it preferable over adding a new option (and am not
>>> completely sure if it is a misuse).
>>> Other have suggested (mis-)using some other option like qscale or
>>> md_decision but that seems like a complete misuse to me.
>>> What are your opinions?
>> What's wrong with the existing AVCodecContext.compression_level?
> 
> I didn't know about it (and assumed nothing exactly fitting existed
> since it was not used).
> So is attached patch ok?
> 
> Greetings,
> Reimar D?ffinger
> 

Hi, why not always use the best compression ?

MvH
Benjamin Larsson




More information about the ffmpeg-devel mailing list