From: "Joseph S. Myers" <joseph@codesourcery.com>
To: Nick Clifton <nickc@redhat.com>
Cc: Tristan Gingold <gingold@adacore.com>,
binutils <binutils@sourceware.org>
Subject: Re: [patch] Convert coff-rs6000.c to iso-c
Date: Thu, 21 Apr 2011 10:22:00 -0000 [thread overview]
Message-ID: <Pine.LNX.4.64.1104211016590.9614@digraph.polyomino.org.uk> (raw)
In-Reply-To: <4DAC3D3A.7090106@redhat.com>
On Mon, 18 Apr 2011, Nick Clifton wrote:
> PS. If you are going to do more of these you might also like to replace
> occurrences of the PTR macro with "void *".
How about declaring all the macros in ansidecl.h that are for pre-C90
compatibility obsolete and removing their definitions once all uses in the
gcc and src repositories are gone? That's ANSI_PROTOTYPES PTR PTRCONST
LONG_DOUBLE PARAMS VPARAMS VA_START VA_OPEN VA_CLOSE VA_FIXEDARG CONST
VOLATILE SIGNED PROTO EXFUN DEFUN DEFUN_VOID AND DOTS NOARGS (plus
definitions as macros of const volatile signed). Some of these macros may
already have been eliminated (GCC's system.h poisons eight of them, for
example, and some more are not defined if IN_GCC) and despite the comments
on ansidecl.h claiming to be from the GNU C Library, the glibc copy was
removed in 1997 so it's purely a libiberty header now.
--
Joseph S. Myers
joseph@codesourcery.com
next prev parent reply other threads:[~2011-04-21 10:22 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-18 12:21 Tristan Gingold
2011-04-18 13:31 ` Nick Clifton
2011-04-18 14:00 ` Tristan Gingold
2011-04-21 10:22 ` Joseph S. Myers [this message]
2011-04-21 10:28 ` Tristan Gingold
2011-04-21 14:08 ` Nick Clifton
2011-04-22 8:02 ` [patch] Convert coff64-rs6000.c " Tristan Gingold
2011-04-22 8:19 ` Andreas Schwab
2011-04-22 8:24 ` Tristan Gingold
2011-04-26 8:47 ` Richard Sandiford
2011-04-26 9:14 ` Tristan Gingold
2011-04-22 8:10 ` [patch] Convert aix386-core " Tristan Gingold
2011-04-26 8:49 ` Richard Sandiford
2011-04-27 9:44 ` Nick Clifton
2011-04-21 21:22 ` [patch] Convert coff-rs6000.c " Ian Lance Taylor
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=Pine.LNX.4.64.1104211016590.9614@digraph.polyomino.org.uk \
--to=joseph@codesourcery.com \
--cc=binutils@sourceware.org \
--cc=gingold@adacore.com \
--cc=nickc@redhat.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).