Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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


  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