Remove windows_process_info::id
authorTom Tromey <tromey@adacore.com>
Thu, 28 Apr 2022 19:40:37 +0000 (13:40 -0600)
committerTom Tromey <tromey@adacore.com>
Fri, 13 May 2022 14:21:16 +0000 (08:21 -0600)
I noticed that windows_process_info::id is only used by gdbserver, and
not really necessary.  This patch removes it.

gdb/nat/windows-nat.h
gdbserver/win32-low.cc

index 27b653674c31a8f3ea06e81114761ac9340163ce..9f7f8b631925e778c04e0b1f032b5239485144d2 100644 (file)
@@ -144,7 +144,6 @@ struct windows_process_info
 {
   /* The process handle */
   HANDLE handle = 0;
-  DWORD id = 0;
   DWORD main_thread_id = 0;
   enum gdb_signal last_sig = GDB_SIGNAL_0;
 
index 192ea465e69c0bbf46375d715c88f639e84671df..5b91ab768e51f20089f5df135c6b79dd29cd57a9 100644 (file)
@@ -337,7 +337,6 @@ do_initial_child_stuff (HANDLE proch, DWORD pid, int attached)
 
   windows_process.last_sig = GDB_SIGNAL_0;
   windows_process.handle = proch;
-  windows_process.id = pid;
   windows_process.main_thread_id = 0;
 
   soft_interrupt_requested = 0;
@@ -678,12 +677,12 @@ win32_process_target::create_inferior (const char *program,
 
   /* Wait till we are at 1st instruction in program, return new pid
      (assuming success).  */
-  cs.last_ptid = wait (ptid_t (windows_process.id), &cs.last_status, 0);
+  cs.last_ptid = wait (ptid_t (pi.dwProcessId), &cs.last_status, 0);
 
   /* Necessary for handle_v_kill.  */
-  signal_pid = windows_process.id;
+  signal_pid = pi.dwProcessId;
 
-  return windows_process.id;
+  return pi.dwProcessId;
 }
 
 /* Attach to a running process.
@@ -816,7 +815,7 @@ win32_process_target::detach (process_info *process)
   resume.sig = 0;
   this->resume (&resume, 1);
 
-  if (!DebugActiveProcessStop (windows_process.id))
+  if (!DebugActiveProcessStop (process->pid))
     return -1;
 
   DebugSetProcessKillOnExit (FALSE);
@@ -1375,7 +1374,7 @@ win32_process_target::write_memory (CORE_ADDR memaddr,
 void
 win32_process_target::request_interrupt ()
 {
-  if (GenerateConsoleCtrlEvent (CTRL_BREAK_EVENT, windows_process.id))
+  if (GenerateConsoleCtrlEvent (CTRL_BREAK_EVENT, signal_pid))
     return;
 
   /* GenerateConsoleCtrlEvent can fail if process id being debugged is