[MPlayer-dev-eng] [PATCH] Remove #if LIBAV*_BUILD >= XXXX everywhere
Michael Niedermayer
michaelni at gmx.at
Wed Jul 12 20:54:29 CEST 2006
Hi
On Wed, Jul 12, 2006 at 01:45:33PM +0200, Diego Biurrun wrote:
> On Wed, Jul 12, 2006 at 11:05:38AM +0200, Michael Niedermayer wrote:
> >
> > On Wed, Jul 12, 2006 at 12:50:17AM +0200, Diego Biurrun wrote:
> > > It's a jungle out there!
> > >
> > > All the code that interfaces with FFmpeg is littered with
> > >
> > > #if LIBAVFORMAT_BUILD >= 4624
> > >
> > > and similar statements. While this may have made sense in the past when
> > > lavc/lavf was not checked out as external, it just clutters the sources
> > > now IMO.
> > >
> > > This may make sense for shared lavc/lavf (which is discouraged anyway),
> > > but even then we could just #error out below a certain version number
> > > instead of disabling some features. To the outside world the disabling
> > > will most probably appear random anyway.
> > >
> > > I propose ripping it all out, patch is attached.
> > >
> > > In any case, the diffstat summary is quite convincing IMO:
> > >
> > > 5 files changed, 8 insertions(+), 325 deletions(-)
> > >
> > > Did I overlook something? Michael, OK to commit?
> >
> > not without hearing a comment from who originally wanted these checks
> > (IIRC it was arpi)
>
> Umm, do you really think Arpi is going to comment on this? He does not
> follow this mailing list, I doubt he is even subscribed. I'd be glad to
> hear the original reason for this, but I don't think the fate of this
> patch should depend on a comment that will probably never arrive...
well my idea was that he should get a chance to comment, if he doesnt
awnser (=doesnt care) than ive no objections to the change, OTOH if he
thinks these checks should stay then iam not agreeing with their removial
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is
More information about the MPlayer-dev-eng
mailing list