Remove linux-waitpid.c debugging code
authorTom Tromey <tom@tromey.com>
Mon, 6 May 2019 00:28:13 +0000 (18:28 -0600)
committerTom Tromey <tom@tromey.com>
Tue, 11 Jun 2019 13:41:00 +0000 (07:41 -0600)
The debugging code in linux-waitpid.c is one of the few remaining
spots that depends on the gdb/gdbserver difference.

My first thought was that this code is not extremely useful, so this
patch removes this code.  (However, if it is actually useful to
someone, we could make it work by introducing a new abstraction.)

gdb/ChangeLog
2019-06-11  Tom Tromey  <tom@tromey.com>

* nat/linux-waitpid.c: Don't include server.h.
(linux_debug): Remove.
(my_waitpid): Update.

gdb/ChangeLog
gdb/nat/linux-waitpid.c

index 8f310f9c641d17fa419672441a2484816db09116..a08f48f84577a5b8e5c2039ba6beb7bf4dc9c802 100644 (file)
@@ -1,3 +1,9 @@
+2019-06-11  Tom Tromey  <tom@tromey.com>
+
+       * nat/linux-waitpid.c: Don't include server.h.
+       (linux_debug): Remove.
+       (my_waitpid): Update.
+
 2019-06-11  Tom Tromey  <tromey@adacore.com>
 
        * infcall.c (_initialize_infcall): Remove trailing newline from
index a7d11ab8d32aa48ec5dbe7f45031806f16981bb8..298032dff9ab0b5769d3c01fea8d47165f4da701 100644 (file)
 
 #include "common/common-defs.h"
 
-#ifdef GDBSERVER
-/* FIXME: server.h is required for the definition of debug_threads
-   which is used in the gdbserver-specific debug printing in
-   linux_debug.  This code should be made available to GDB also,
-   but the lack of a suitable flag to enable it prevents this.  */
-#include "server.h"
-#endif
-
 #include "linux-nat.h"
 #include "linux-waitpid.h"
 #include "common/gdb_wait.h"
 
-/* Print debugging output based on the format string FORMAT and
-   its parameters.  */
-
-static inline void ATTRIBUTE_PRINTF (1,2)
-linux_debug (const char *format, ...)
-{
-#ifdef GDBSERVER
-  if (debug_threads)
-    {
-      va_list args;
-      va_start (args, format);
-      debug_vprintf (format, args);
-      va_end (args);
-    }
-#endif
-}
-
 /* Convert wait status STATUS to a string.  Used for printing debug
    messages only.  */
 
@@ -79,20 +54,13 @@ status_to_str (int status)
 int
 my_waitpid (int pid, int *status, int flags)
 {
-  int ret, out_errno;
-
-  linux_debug ("my_waitpid (%d, 0x%x)\n", pid, flags);
+  int ret;
 
   do
     {
       ret = waitpid (pid, status, flags);
     }
   while (ret == -1 && errno == EINTR);
-  out_errno = errno;
-
-  linux_debug ("my_waitpid (%d, 0x%x): status(%x), %d\n",
-              pid, flags, (ret > 0 && status != NULL) ? *status : -1, ret);
 
-  errno = out_errno;
   return ret;
 }