From: Kevin Buettner <kevinb@redhat.com>
To: David Carlton <carlton@math.stanford.edu>,
gdb-patches@sources.redhat.com
Cc: Jim Blandy <jimb@red-bean.com>, Elena Zannoni <ezannoni@redhat.com>
Subject: Re: [rfc] split up symtab.h
Date: Tue, 08 Oct 2002 13:54:00 -0000 [thread overview]
Message-ID: <1021008205446.ZM16832@localhost.localdomain> (raw)
In-Reply-To: David Carlton <carlton@math.stanford.edu> "[rfc] split up symtab.h" (Oct 8, 1:14pm)
On Oct 8, 1:14pm, David Carlton wrote:
> I'm sick of having to recompile half of GDB every time I touch
> symtab.h. There's lots of different things in that file; it's
> included in 137 different places (counting only the gdb directory, not
> gdb/mi, etc.), but there's no one thing that it defines that is used
> in more than 71 places, and a lot of things that it defines are used
> in a lot fewer places than that.
[...]
> Anyways, if anybody else is similarly annoyed with symtab.h then I'll
> try to split things up and make a more concrete RFA in a bit.
Aside from the build time issue, are there other reasons why splitting
up symtab.h is desirable?
Here are several reasons for not splitting it:
1) The list of includes for many .c files will (I suspect) grow
quite a bit. If it turns out that you'll be replacing one
#include statement with five or size (per source file), I
can't really see that making the split was an advantage.
2) One could argue that modifying symtab.h *should* be a heavy weight
operation. I.e, you're modifying something that's at the very
heart of gdb and you need to take great care.
3) Makefile.in maintenance becomes harder due to the larger
number of header files.
I should note that I don't find any of the above reasons to be
overly compelling. I just think that we need a better reason
for making such a split than the build time consideration.
Kevin
next prev parent reply other threads:[~2002-10-08 20:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-10-08 13:14 David Carlton
2002-10-08 13:54 ` Kevin Buettner [this message]
2002-10-08 15:05 ` David Carlton
2002-10-08 15:11 ` Michael Snyder
2002-10-08 15:22 ` David Carlton
2002-10-18 14:12 ` Elena Zannoni
2002-10-18 14:52 ` David Carlton
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=1021008205446.ZM16832@localhost.localdomain \
--to=kevinb@redhat.com \
--cc=carlton@math.stanford.edu \
--cc=ezannoni@redhat.com \
--cc=gdb-patches@sources.redhat.com \
--cc=jimb@red-bean.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