On 2012-07-02 22:10:19 +0200, Diego Biurrun wrote:
> ---
>  cmdutils.c             |   11 ++++-
>  libavdevice/Makefile   |    3 +-
>  libavdevice/avdevice.c |    2 +
>  libavdevice/avdevice.h |   12 -----

misses version.h include, this is an externally visible change

>  libavdevice/version.h  |   42 ++++++++++++++++++
>  libavutil/Makefile     |    1 +
>  libavutil/avutil.h     |   86 ------------------------------------

same

>  libavutil/eval.c       |    1 +
>  libavutil/fifo.h       |    4 +-
>  libavutil/file.c       |    2 +
>  libavutil/imgutils.c   |    1 +
>  libavutil/opt.h        |    1 +
>  libavutil/samplefmt.h  |    1 +
>  libavutil/utils.c      |    1 +
>  libavutil/version.h    |  113 
> ++++++++++++++++++++++++++++++++++++++++++++++++
>  libswscale/Makefile    |    3 +-
>  libswscale/swscale.h   |   26 +-----------
>  libswscale/utils.c     |    1 +
>  libswscale/version.h   |   54 +++++++++++++++++++++++
>  19 files changed, 237 insertions(+), 128 deletions(-)

configure changes missing

>  create mode 100644 libavdevice/version.h
>  create mode 100644 libavutil/version.h
>  create mode 100644 libswscale/version.h

I'm not opposing creating version.h headers but please make sure
they are included from the main library header and I think we
should guarantee that.

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to