This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: Initialization of hpux_threads
- To: Andrew Cagney <ac131313 at cygnus dot com>
- Subject: Re: Initialization of hpux_threads
- From: Jimmy Guo <guo at cup dot hp dot com>
- Date: Tue, 21 Mar 2000 16:09:13 -0800 (PST)
- Cc: Michael Snyder <msnyder at cygnus dot com>,John David Anglin <dave at hiauly1 dot hia dot nrc dot ca>,gdb-patches at sourceware dot cygnus dot 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