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

Stefano Sabatini stefano.sabatini-lala
Sun Jun 22 00:38:29 CEST 2008


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.

Rationale: it would be nice to expose abbreviations to the user, for
example in show_formats().

A theoretically better solution could be to use AVOption to define the
frame rate and frame size named constants as suggested by Michael more
than one year ago, but this doesn't seem a trivial deed since it
requires a deep rethinking of AVOption and of the API, anyway we can
get rid of AV**Abbr when we'll eventually get to accomplish that.

Kind regards.
-- 
FFmpeg = Furious Fostering Monstrous Portentous Enchanting Gorilla
-------------- next part --------------
A non-text attachment was scrubbed...
Name: export-video-framerate-size-abbrs-00.patch
Type: text/x-diff
Size: 3808 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080622/09d1758c/attachment.patch>



More information about the ffmpeg-devel mailing list