[FFmpeg-devel] [PATCH] Export video framerates and sizes abbreviations

Stefano Sabatini stefano.sabatini-lala
Sun Jun 22 10:03:27 CEST 2008


On date Sunday 2008-06-22 03:05:33 +0200, Michael Niedermayer encoded:
> On Sun, Jun 22, 2008 at 12:38:29AM +0200, Stefano Sabatini wrote:
> > Hi as in subject.
> > 
> > this is in truth more like a RFC since I would like to hear opinions
> > of other developers.
> > 
> > I wonder we could have another field inside the structs with a more
> > descriptive name (e.g. long_name), in this case the structs could be
> > named AVVideoFrameRateFormat and AVVideoFrameSizeFormat or maybe you
> > prefer it like in the patch or maybe you don't want such things at
> > all.
> 
> Iam against this patch

Hi, 
I respect your decision, just I'd like you to elaborate more on the
reasons for this reject.

The idea was not to have the user read the code to understand which
the various abbrevs are and what they mean, the current solution
(document them in the docs) leads to code duplication.

Regards.
-- 
FFmpeg = Foolish Faboulous Merciful Pitiless Experimenting Guide




More information about the ffmpeg-devel mailing list