[FFmpeg-devel] [FFmpeg-cvslog] Merge commit '7a745f014f528d1001394ae4d2f4ed1a20bf7fa2'
jamrial at gmail.com
Mon Nov 14 20:19:23 EET 2016
On 11/14/2016 8:01 AM, Michael Niedermayer wrote:
> On Sun, Nov 13, 2016 at 10:32:13PM +0100, Hendrik Leppkes wrote:
>> ffmpeg | branch: master | Hendrik Leppkes <h.leppkes at gmail.com> | Sun Nov 13 22:29:04 2016 +0100| [bd0db4a32d85d027da4d4dc00490c20048090312] | committer: Hendrik Leppkes
>> Merge commit '7a745f014f528d1001394ae4d2f4ed1a20bf7fa2'
>> * commit '7a745f014f528d1001394ae4d2f4ed1a20bf7fa2':
>> options_table: Add aliases for color properties
>> Merged-by: Hendrik Leppkes <h.leppkes at gmail.com>
>> libavcodec/options_table.h | 32 ++++++++++++++++++++++++--------
>> libavcodec/version.h | 2 +-
>> 2 files changed, 25 insertions(+), 9 deletions(-)
> Please dont forget to add documentation for every option you add.
> IIRC Libav has code to automatically generate documentation from
> AVOptions. Several FFmpeg developers objected against enabling that
> code. IIRC because hand written documentation is of much higher quality
Automatically generated documentation for options like this sounds very
useful. I don't know why it was rejected. There's not much more "quality"
to be had over manually written lines for AVOptions.
> You could of course also try to ping the people previously objecting to
> auto generated docs, maybe they would be interrested in helping.
> If there is interrest i can also attempt to write a fate test to
> detect newly added options without docs?
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel