Remove unnecessary returns and unused variables in AIX.

This is a patch to simplify the pd_activate () in aix-thread.c incase of a failure to start a thread session
, since pd_activate () now has return type void.

Also this patch fixes the shadow declarartion warning in sync-threadlists.

diff --git a/gdb/aix-thread.c b/gdb/aix-thread.c
index c8147defc046418570dd216b791efe8e09141ce4..3eb8f57422caf6c1a6053486eee28a086b8e080c 100644 (file)
--- a/gdb/aix-thread.c
+++ b/gdb/aix-thread.c
@@ -829,7 +829,6 @@ sync_threadlists (pid_t pid)
   pthread_t pthid;
   pthdb_tid_t tid;
   process_stratum_target *proc_target = current_inferior ()->process_target ();
-  thread_info  *tp;
   struct aix_thread_variables *data;
   data = get_thread_data_helper_for_pid (pid);
 
@@ -931,7 +930,7 @@ sync_threadlists (pid_t pid)
 
              if (gptid.is_pid ())
                {
-                 tp = proc_target->find_thread (gptid);
+                 thread_info *tp = proc_target->find_thread (gptid);
                  thread_change_ptid (proc_target, gptid, pptid);
                  aix_thread_info *priv = new aix_thread_info;
                  priv->pdtid = pbuf[pi].pdtid;
@@ -1027,12 +1026,8 @@ pd_activate (pid_t pid)
   status = pthdb_session_init (pid, data->arch64 ? PEM_64BIT : PEM_32BIT,
                               PTHDB_FLAG_REGS, &pd_callbacks,
                               &data->pd_session);
-  if (status != PTHDB_SUCCESS)
-    {
-      return ptid_t (pid);
-    }
-  data->pd_active = 1;
-  return;
+  if (status == PTHDB_SUCCESS)
+    data->pd_active = 1;
 }
 
 /* AIX implementation of update_thread_list.  */
@@ -1746,7 +1741,6 @@ store_regs_user_thread (const struct regcache *regcache, pthdb_pthread_t pdtid)
   uint64_t int64;
   struct aix_thread_variables *data;
   data = get_thread_data_helper_for_ptid (inferior_ptid);
-  int ret;
   __vmx_context_t vmx;
   __vsx_context_t  vsx;