[FFmpeg-devel] [PATCH] Clean up lib* version definitions
Michael Niedermayer
michaelni
Mon Feb 18 21:53:35 CET 2008
On Mon, Feb 18, 2008 at 08:47:50PM +0000, M?ns Rullg?rd wrote:
> Diego Biurrun <diego at biurrun.de> writes:
>
> > 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.
>
> Well, does anyone else have an opinion?
Iam slightly against it but its not a big thing, if you want it in svn
iam fine with it.
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080218/72fc1713/attachment.pgp>
More information about the ffmpeg-devel
mailing list