[FFmpeg-devel] [PATCH] Clean up lib* version definitions

Diego Biurrun diego
Sun Feb 17 22:46:34 CET 2008


On Sun, Feb 17, 2008 at 07:55:26PM +0000, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Sun, Feb 17, 2008 at 06:14:31PM +0000, Mans Rullgard wrote:
> >> Updating version numbers now requires changing only one place.
> >> ---
> >>  configure                 |    2 +-
> >>  libavcodec/avcodec.h      |    6 ++++--
> >>  libavdevice/avdevice.h    |    6 ++++--
> >>  libavformat/avformat.h    |    6 ++++--
> >>  libavutil/avutil.h        |   12 ++++++++++--
> >>  libpostproc/postprocess.h |    6 ++++--
> >
> >>  6 files changed, 27 insertions(+), 11 deletions(-)
> >
> > Hmm, this more than doubles the number of lines of code, and requires
> 
> The new macros aside, it's replacing two lines with three in each
> header file (plus a blank line I thought looked better).  That's not
> what I call more than double.
> 
> > the reader to understand/guess the meaning for an additional layer of
> > macros.
> 
> I'd be happy to add a comment explaining them.
> 
> > Do you really think this is a good idea?
> 
> People have forgotten to update both sets of version numbers on
> occasion, and there's been a much messier patch posted in the past.
> IMHO, having the same information in two (or more) places is always
> bad.

Being one person who managed to update just one of the two version
numbers in the past: I like Mans' proposal.

Diego




More information about the ffmpeg-devel mailing list