This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH/committed] sim: delete duplicate SIGINT handling


On 02/17/14 15:33, Mike Frysinger wrote:
Many of the simulators change the SIGINT handler.
E.g., moxie/interp.c:

   sigsave = signal (SIGINT, interrupt);

However, this is unnecessary.
remote-sim.h already provides an API for asynchronously stopping
a sim; and both gdb and the drivers (run.c and nrun.c at least,
I didn't check the others) install a SIGINT handler which
calls this method.

URL: https://sourceware.org/bugzilla/show_bug.cgi?id=16450
Reported-by: Tom Tromey <tromey@redhat.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
  sim/h8300/ChangeLog      |  7 +++++++
  sim/h8300/compile.c      | 14 --------------
  sim/mcore/ChangeLog      |  6 ++++++
  sim/mcore/interp.c       | 10 ----------
  sim/microblaze/ChangeLog |  6 ++++++
  sim/microblaze/interp.c  | 10 ----------

OK for MicroBlaze.


--
Michael Eager	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]