From 52554a0e324951b5c58e71f2815e841377e9c50c Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Tue, 19 Mar 2013 09:23:17 -0600 Subject: constify to_detach This patch constifies the target_ops method to_detach. This is a small cleanup, but also, I think, a bug-prevention fix, since gdb already acts as if the "args" argument here was const. In particular, top.c:quit_force calls kill_or_detach via iterate_over_inferiors. kill_or_detach calls target_detach, passing the same argument each time. So, if one of these methods was not const-correct, then kill_or_detach would change its behavior in a strange way. I could not build every target I modified in this patch. I've inspected them all by hand, though. Many targets do not use the "args" parameter; a couple pass it to atoi; and a few pass it on to the to_detach method of the target beneath. The only code that required a real change was in linux-nat.c, and that only needed the introduction of a temporary variable for const-correctness. 2013-11-08 Tom Tromey * aix-thread.c (aix_thread_detach): Update. * corelow.c (core_detach): Update. * darwin-nat.c (darwin_detach): Update. * dec-thread.c (dec_thread_detach): Update. * gnu-nat.c (gnu_detach): Update. * go32-nat.c (go32_detach): Update. * inf-ptrace.c (inf_ptrace_detach): Update. * inf-ttrace.c (inf_ttrace_detach): Update. * linux-fork.c (linux_fork_detach): Update. * linux-fork.h (linux_fork_detach): Update. * linux-nat.c (linux_nat_detach): Update. Introduce "tem" local for const-correctness. * linux-thread-db.c (thread_db_detach): Update. * monitor.c (monitor_detach): Update. * nto-procfs.c (procfs_detach): Update. * procfs.c (procfs_detach): Update. * record.c (record_detach): Update. * record.h (record_detach): Update. * remote-m32r-sdi.c (m32r_detach): Update. * remote-mips.c (mips_detach): Update. * remote-sim.c (gdbsim_detach): Update. * remote.c (remote_detach_1, remote_detach) (extended_remote_detach): Update. * sol-thread.c (sol_thread_detach): Update. * target.c (target_detach): Make "args" const. (init_dummy_target): Update. * target.h (struct target_ops) : Make argument const. (target_detach): Likewise. * windows-nat.c (windows_detach): Update. --- gdb/ChangeLog | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) (limited to 'gdb/ChangeLog') diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1c23f962dcc..4d1c26d0167 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,35 @@ +2013-11-08 Tom Tromey + + * aix-thread.c (aix_thread_detach): Update. + * corelow.c (core_detach): Update. + * darwin-nat.c (darwin_detach): Update. + * dec-thread.c (dec_thread_detach): Update. + * gnu-nat.c (gnu_detach): Update. + * go32-nat.c (go32_detach): Update. + * inf-ptrace.c (inf_ptrace_detach): Update. + * inf-ttrace.c (inf_ttrace_detach): Update. + * linux-fork.c (linux_fork_detach): Update. + * linux-fork.h (linux_fork_detach): Update. + * linux-nat.c (linux_nat_detach): Update. Introduce "tem" + local for const-correctness. + * linux-thread-db.c (thread_db_detach): Update. + * monitor.c (monitor_detach): Update. + * nto-procfs.c (procfs_detach): Update. + * procfs.c (procfs_detach): Update. + * record.c (record_detach): Update. + * record.h (record_detach): Update. + * remote-m32r-sdi.c (m32r_detach): Update. + * remote-mips.c (mips_detach): Update. + * remote-sim.c (gdbsim_detach): Update. + * remote.c (remote_detach_1, remote_detach) + (extended_remote_detach): Update. + * sol-thread.c (sol_thread_detach): Update. + * target.c (target_detach): Make "args" const. + (init_dummy_target): Update. + * target.h (struct target_ops) : Make argument const. + (target_detach): Likewise. + * windows-nat.c (windows_detach): Update. + 2013-11-07 Doug Evans PR 11786 -- cgit v1.2.1