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

[Benjamin Larsson]

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