Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jimmy Guo <guo@cup.hp.com>
To: Andrew Cagney <ac131313@cygnus.com>
Cc: Michael Snyder <msnyder@cygnus.com>,
	John David Anglin <dave@hiauly1.hia.nrc.ca>,
	gdb-patches@sourceware.cygnus.com
Subject: Re: Initialization of hpux_threads
Date: Tue, 21 Mar 2000 15:09:00 -0000	[thread overview]
Message-ID: <Pine.LNX.4.10.10003211552360.31590-100000@hpcll168.cup.hp.com> (raw)
In-Reply-To: <38D7F9C4.ECF7E17F@cygnus.com>

I was looking at an old source tree and didn't see its 'revival'.
Yes it is some setup to solve the kinds of problem you mentioned.
But any other usage of it will only introduce multiple calls to
a _initialize_* routine.

The initializer in hpux-thread.c is just one of these cases.  They're
mostly fixed as of today's tree except for a remaining one -
remote-nrom.c.

- Jimmy Guo, guo@cup.hp.com


On Wed, 22 Mar 2000, Andrew Cagney wrote:

>> This looks fine.  Just an observation:
>> CONFIG_OBS is part of DEPFILES, which is part of COMMON_OBS, which is
>> part of OBS, which is part of INITFILES.  It looks like all three usages
>> of CONFIG_INITS in configure.in can be removed.  Maybe you can just
>> remove CONFIG_INITS altogether in configure.in and Makefile.in.  Since
>> OBS file list is already looked at by init.c rule, it really makes no
>> sense to maintain a CONFIG_INITS file list.
>
>No. CONFIG_INITS was added because things get complicated when adding
>sub-directory files (eg MI) that contain _initialize_*() functions.  The
>mistake was to assume that it was needed in all (rather than special)
>cases.
>
>By doing this the MI avoided the need to add any nasty initialization
>hooks to main() - unlike tui, MPW, and GDBTK.
>
>	Andrew


       reply	other threads:[~2000-03-21 15:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <38D7F9C4.ECF7E17F@cygnus.com>
2000-03-21 15:09 ` Jimmy Guo [this message]
2000-03-16 13:53 John David Anglin
     [not found] ` <38D2CB7D.24B6@cygnus.com>
2000-03-19 22:41   ` Andrew Cagney

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.10.10003211552360.31590-100000@hpcll168.cup.hp.com \
    --to=guo@cup.hp.com \
    --cc=ac131313@cygnus.com \
    --cc=dave@hiauly1.hia.nrc.ca \
    --cc=gdb-patches@sourceware.cygnus.com \
    --cc=msnyder@cygnus.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