[FFmpeg-devel] [FFmpeg-cvslog] rawdec: factor video demuxer definitions into a macro.

Mike Scheutzow mjs973 at optonline.net
Sun May 29 21:14:02 CEST 2011


Anton Khirnov wrote:
> ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Wed May 25 08:14:13 2011 +0200| [fa4924a35818b2564050119ed7c14cbdd2b56065] | committer: Anton Khirnov
>
> rawdec: factor video demuxer definitions into a macro.
>
>   
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=fa4924a35818b2564050119ed7c14cbdd2b56065
>>     
> ---
>
>  libavformat/cavsvideodec.c |   11 +----------
>  libavformat/diracdec.c     |   11 +----------
>  libavformat/dnxhddec.c     |   11 +----------
>  libavformat/h261dec.c      |   12 +-----------
>  libavformat/h263dec.c      |   12 +-----------
>  libavformat/h264dec.c      |   12 +-----------
>  libavformat/m4vdec.c       |   12 +-----------
>  libavformat/mpegvideodec.c |   11 +----------
>  libavformat/rawdec.c       |   23 ++---------------------
>  libavformat/rawdec.h       |   12 ++++++++++++
>  10 files changed, 22 insertions(+), 105 deletions(-)
> ...
>
>   

This patch feels like a step backwards to me.

It is simply documentation removal, for no benefit.

How is it better that developers have to decode another level of 
indirection when reading source code?

Mike Scheutzow




More information about the ffmpeg-devel mailing list