[FFmpeg-devel] [PATCH 1/6] lavf: move the version macros to a new header

Luca Barbato lu_zero
Fri Jan 21 20:41:07 CET 2011


On 01/21/2011 08:31 PM, M?ns Rullg?rd wrote:
> Anton Khirnov <anton at khirnov.net> writes:
> 
>> ---
>>  libavformat/Makefile   |    2 +-
>>  libavformat/avformat.h |   64 +---------------------------------
>>  libavformat/avio.h     |    2 +
>>  libavformat/version.h  |   90 ++++++++++++++++++++++++++++++++++++++++++++++++
>>  4 files changed, 94 insertions(+), 64 deletions(-)
>>  create mode 100644 libavformat/version.h
> 
> Looks good, but I'd like a second opinion before pushing.

Looks fine, but why avio.h is public and included in avformat.h as
#include "avio.h" ?

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list