[Ffmpeg-devel] [PATCH] Enhance genre, year and track tag support

[Panagiotis Issaris]

Hi,

Op dinsdag 12 september 2006 20:19, schreef Michael Niedermayer:
>[...]
> while i dont know if we will or will not need some init function, i somehow
> think that it that it would be best to not do any scaling but instead
> support SI postfixes (k,m,g) and maybe bit/byte too ...
Good idea. I'll post a patch adding this soon. 

> > * Some AVOption names are a lot longer then the corresponding OptionDef names. For example 
> > the parameter for setting the video bitrate would change from "b" to "bit_rate". I think
> > this would annoy a lot of users. Are the AVOption parameter names already used? By external
> > projects like MPlayer or VLC? If not, I could start renaming the AVOption names to the shorter
> > ffmpeg.c/OptionDef names.
> 
> yes, rename them
Excellent :) I'll post a patch doing this soon.

> > * The AVOption default values differ from the defaults given to the same parameters in the
> > ffmpeg.c file. I think it would be best to not change the current behavior and so take
> > the defaults from the ffmpeg.c file, right?
> 
> probably ...


With friendly regards,
Takis