Remove unnecessary call to get_thread_db_info

In thread_db_detach, we call get_thread_db_info to first check if there
exists a thread_db_info entry for the pid to detach.  If there is, then
we call delete_thread_db_info.  It's unnecessary to call
get_thread_db_info in the first place, since delete_thread_db_info
handles the case where no thread_db_info entry exist for the given pid.

gdb/ChangeLog:

	* linux-thread-db.c (thread_db_detach): Remove call to
	delete_thread_db_info.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8555b55afd8cf23647453d9d86a8f130c050902c..bba278e1d665c1fdc3c96250d6292771fb03654b 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2017-12-29  Simon Marchi  <simon.marchi@ericsson.com>
+
+       * linux-thread-db.c (thread_db_detach): Remove call to
+       delete_thread_db_info.
+
 2017-12-28  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * target.h (enum target_object) <TARGET_OBJECT_HPUX_UREGS,

diff --git a/gdb/linux-thread-db.c b/gdb/linux-thread-db.c
index 27f9ce8e593a9d00fb3f7eb68ce590e8d74c2451..7fe6e2d87118317ae1481eecf02d6c90991c1551 100644 (file)
--- a/gdb/linux-thread-db.c
+++ b/gdb/linux-thread-db.c
@@ -1093,12 +1093,8 @@ static void
 thread_db_detach (struct target_ops *ops, const char *args, int from_tty)
 {
   struct target_ops *target_beneath = find_target_beneath (ops);
-  struct thread_db_info *info;
-
-  info = get_thread_db_info (ptid_get_pid (inferior_ptid));
 
-  if (info)
-    delete_thread_db_info (ptid_get_pid (inferior_ptid));
+  delete_thread_db_info (ptid_get_pid (inferior_ptid));
 
   target_beneath->to_detach (target_beneath, args, from_tty);