This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: (patch) hpjyg15a, was Re: (patch) hpjyg15: hppah-nat.c & related
- To: Stan Shebs <shebs at cygnus dot com>
- Subject: Re: (patch) hpjyg15a, was Re: (patch) hpjyg15: hppah-nat.c & related
- From: Andrew Cagney <ac131313 at cygnus dot com>
- Date: Mon, 22 Nov 1999 22:39:03 +1100
- CC: Jimmy Guo <guo at cup dot hp dot com>, gdb-patches at sourceware dot cygnus dot com
- Organization: Cygnus Solutions
- References: <Pine.LNX.4.10.9911111640190.6426-100000@hpcll168.cup.hp.com>
Stan,
Here is the infrun.c part of HP's patch 15.
Jimmy Guo wrote:
>
> >I'd prefer the name (target_signal_p()) (Well actually I don't prefer
> >that name but its more consistent with the general naming convetion :-).
> >
> >I'll check in a change to target.[ch] that adds the target_singal_p().
> >Hmm, I might do it slightly differently - were getting too many separate
> >places where there is code like:
> >
> > #if SIG...
> > case TARGET_SIGNAL...
> > #endif
> >
> >With that in, can the hppa-nat.c be re-submitted? BTW, you might also
> >split it, I think it contains two changes - the code that uses this new
> >function and something else.
>
> Sure. Here is the part which makes use of target_signal_p (the
> target.[ch] changes you are making).
>
> Ignore the hpjyg15 patch, but use this (hpjyg15a), and the sequel I'm
> cooking (hpjug15b).
>
> ChangeLog:
>
> 1999-11-08 Jimmy Guo <guo@cup.hp.com>
>
> * infrun.c (handle_command): On HP, call
> require_notification_of_events to ignore signals we don't
> care.
>
> Index: gdb/infrun.c
> /opt/gnu/bin/diff -r -c -N /view/guo.wdb.c//CLO/Components/WDB/Src/gnu/gdb/infrun.c gdb/infrun.c
> *** /view/guo.wdb.c//CLO/Components/WDB/Src/gnu/gdb/infrun.c Fri Nov 5 18:37:38 1999
> --- gdb/infrun.c Fri Nov 5 18:38:41 1999
> ***************
> *** 3621,3626 ****
> --- 3621,3633 ----
> }
> }
>
> +
> + #ifdef GDB_TARGET_IS_HPPA
> + /* RM: Use OS interface to ignore signals we don't care about */
> + if (target_has_execution)
> + require_notification_of_events (inferior_pid);
> + #endif
> +
> do_cleanups (old_chain);
> }
>
Andrew