Re: Continued tidying up of ltdl

[Gary V. Vaughan]

On Sun, Nov 19, 2000 at 11:27:10PM -0200, Alexandre Oliva wrote:
> On Nov 19, 2000, "Gary V. Vaughan" <address@hidden> wrote:
> 
> >> I can't think of any way to get deprecation warnings either.  Except
> >> by renaming the header file and adding a `#warn' line to the old
> >> header.  Ugh!
> 
> > Ah.  That gives me an idea... how about this?
> 
> *blush*  Nah!  Too obvious :-)  
> 
> Hmm...  Except that the `#' or `#warning' should be indented, so that
> preprocessors that don't understand `#warning' don't complain about
> it.  Also, it would probably be a good idea to enclose this whole
> thing in `#ifdef __STDC__'.

Hmmm?  The #warning?  Is that an ANSI preprocessor only thing then?

> But the more I think of it, the more I think we shouldn't enable the
> backward-compatible names by default.  After all, someone who switches
> to libltdl in their sources will hopefully notice the problem
> immediately, and either defer the upgrade, enable the
> backward-compatibility flag or adjust his/her sources to use the new
> names.  Then, there's no need for a soon-to-become-useless -Define to
> enable the new names.

Agreed.  How's this then? (see attached)

> Pre-installed libltdl may have this problem, though.  Nothing that
> autoconf couldn't take care of.

I'm not sure that it is worth worrying about this until we get an
actual report.  I'd be surprised if there is anyone brave enough to
rely on an installed libltdl.  And as you say, they can always whip up
an autoconf macro =)O|

Cheers,
        Gary.
-- 
  ___              _   ___   __              _         mailto: address@hidden
 / __|__ _ _ ___ _| | / / | / /_ _ _  _ __ _| |_  __ _ ___       address@hidden 
| (_ / _` | '_|// / |/ /| |/ / _` | || / _` | ' \/ _` | _ \
 \___\__,_|_|\_, /|___(_)___/\__,_|\_,_\__, |_||_\__,_|//_/
home page:  /___/                      /___/                  gpg public key:
http://www.oranda.demon.co.uk           http://www.oranda.demon.co.uk/key.asc

2000-11-20-posix-namespace.patch
Description: Text document