all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: dhruva <dhruvakm@gmail.com>
Cc: sridhar_ml@yahoo.com, emacs-devel@gnu.org, jasonr@gnu.org
Subject: Re: 23.0.50 compile problem on Windows XP
Date: Fri, 05 Oct 2007 12:16:10 +0200	[thread overview]
Message-ID: <ufy0p989h.fsf@gnu.org> (raw)
In-Reply-To: <e3f230850710050300ke748ca5m43847769a5ecf7ac@mail.gmail.com> (message from dhruva on Fri, 5 Oct 2007 15:30:58 +0530)

> Date: Fri, 5 Oct 2007 15:30:58 +0530
> From: dhruva <dhruvakm@gmail.com>
> Cc: "Jason Rumney" <jasonr@gnu.org>, sridhar_ml@yahoo.com, emacs-devel@gnu.org
> 
> On 10/5/07, Eli Zaretskii <eliz@gnu.org> wrote:
> > I still think that we should avoid a macro with a name that's
> > identical to a standard function.  We should rename it to something
> > that cannot clash with the standard C library.
> 
> If we are trying to overcome a limitation, IMHO, we are forced to
> redifine the macro with the same name, else, we will have to make
> changes in the code to use a different name.

Well, of course I meant to modify all the callers of `abs' to use the
alternative name!  There are about 65 places in 13 source files where
`abs' is used; renaming them all is a trivially simple job.

> Samba project has done this sort of a thing by using function pointers
> to all (almost) system API functions. On platforms which have those
> APIs, they point to the system provided function and on platforms
> where they are not, we implement them in a shared library (to delay
> the binding and provide flexibility by just replacing the shared
> library/implementation).

I don't see how this experience is relevant: we are talking about a
macro, not a function, and a macro whose name is reserved for a
standard library function, on top of that.

  reply	other threads:[~2007-10-05 10:16 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-02 22:42 23.0.50 compile problem on Windows XP Sridhar Boovaraghavan
2007-10-03  8:00 ` Eli Zaretskii
2007-10-03  8:16   ` dhruva
2007-10-03  8:23     ` Jason Rumney
2007-10-03 10:01       ` dhruva
2007-10-03 12:10         ` Jason Rumney
2007-10-03 12:20           ` Jason Rumney
2007-10-03 12:59             ` dhruva
2007-10-04 18:03             ` Sridhar Boovaraghavan
2007-10-04 22:11               ` Jason Rumney
2007-10-05  0:46                 ` David Robinow
2007-10-05  8:02                   ` Eli Zaretskii
2007-10-05  1:17                 ` Sridhar Boovaraghavan
2007-10-05  8:06                   ` Eli Zaretskii
2007-10-05  8:00                 ` Eli Zaretskii
2007-10-05  8:11                   ` Jason Rumney
2007-10-05  8:30                     ` Eli Zaretskii
2007-10-06 13:14                       ` Richard Stallman
2007-10-13 12:49                         ` Eli Zaretskii
2007-10-05  9:31                 ` Eli Zaretskii
2007-10-05 10:00                   ` dhruva
2007-10-05 10:16                     ` Eli Zaretskii [this message]
2007-10-05  7:56           ` Eli Zaretskii
2007-10-05  8:08             ` Jason Rumney
2007-10-05  8:19               ` Eli Zaretskii
2007-10-05  8:21             ` dhruva
2007-10-05  8:31               ` Eli Zaretskii
2007-10-05  8:53                 ` dhruva
2007-10-03 12:19         ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ufy0p989h.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=dhruvakm@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=jasonr@gnu.org \
    --cc=sridhar_ml@yahoo.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.