[Ffmpeg-devel] AVOptions

Rich Felker dalias
Mon Sep 12 22:38:54 CEST 2005


On Mon, Sep 12, 2005 at 11:53:23AM -0500, Joshua Varner wrote:
> On 9/12/05, Rich Felker <dalias at aerifal.cx> wrote:
> > On Mon, Sep 12, 2005 at 05:59:08PM +0200, Luca Barbato wrote:
> > > Michael Niedermayer wrote:
> > > > Hi
> > > >
> > > > On Sun, Sep 11, 2005 at 01:18:40PM +0200, Michael Niedermayer wrote:
> > > >
> > > >>Hi
> > > >>
> > > >>as everyone certainly already knows ffmpeg/libav* has a nice&new AVOption
> > > >>API, comments & suggestions for improvements are welcome
> > > >>
> > > >>and dont complain in a month that its not useable in your application, if
> > > >>you know of an issue or have a suggestion to improve it then post it now
> > > >
> > > >
> > > > and if anyone has suggestions for better names for some options now is the
> > > > right time as compatibility is slightly broken anyway
> > > >
> > > > and a few specific questions
> > > > currently all AVOption constants are upper case so for example '-idct SIMPLE'
> > > > is that ok, or do people want lowercase for them?
> > >
> > > I guess lowercase are more readable but is just a matter of taste.
> > 
> > i think so too. i find uppercase painful to read, and more tedious to
> > type (and it looks like you're chatting on aol...)
> > 
> 
> Why not make them case insensitive, Option processing will only
> happen once so it's not expensive. Does anyone think they'll ever
> want "SIMPLE" and "simple" to be different options?

well this will require special comparison functions to ensure that
case mapping behaves according to ascii only -- otherwise we'll have
problems when people set it to turkish locale and such... :(

rich





More information about the ffmpeg-devel mailing list