Avoid conflict with gnulib open/close macros.
authorRoland McGrath <mcgrathr@google.com>
Wed, 2 Mar 2022 00:03:58 +0000 (16:03 -0800)
committerRoland McGrath <mcgrathr@google.com>
Thu, 3 Mar 2022 19:21:36 +0000 (11:21 -0800)
On some systems, the gnulib configuration will decide to define open
and/or close as macros to replace the POSIX C functions.  This
interferes with using those names in C++ class or namespace scopes.

gdbsupport/
* event-pipe.cc (event_pipe::open): Renamed to ...
(event_pipe::open_pipe): ... this.
(event_pipe::close): Renamed to ...
(event_pipe::close_pipe): ... this.
* event-pipe.h (class event_pipe): Updated.
gdb/
* inf-ptrace.h (async_file_open, async_file_close): Updated.
gdbserver/
* gdbserver/linux-low.cc (linux_process_target::async): Likewise.

gdb/inf-ptrace.h
gdbserver/linux-low.cc
gdbsupport/event-pipe.cc
gdbsupport/event-pipe.h

index 62cc7778767565793f458ced93975a26925fef17..8f18d4579a6dbf5f156174048535e5dabb0074ec 100644 (file)
@@ -77,9 +77,9 @@ struct inf_ptrace_target : public inf_child_target
 protected:
   /* Helper routines for interacting with the async event pipe.  */
   bool async_file_open ()
-  { return m_event_pipe.open (); }
+  { return m_event_pipe.open_pipe (); }
   void async_file_close ()
-  { m_event_pipe.close (); }
+  { m_event_pipe.close_pipe (); }
   void async_file_flush ()
   { m_event_pipe.flush (); }
   void async_file_mark ()
index 301e42a36f36daf59501b4d7e998b2479ed3a9ba..0a5b60631044afbc56313f8057a0fdde5bd04f20 100644 (file)
@@ -5810,7 +5810,7 @@ linux_process_target::async (bool enable)
 
       if (enable)
        {
-         if (!linux_event_pipe.open ())
+         if (!linux_event_pipe.open_pipe ())
            {
              gdb_sigmask (SIG_UNBLOCK, &mask, NULL);
 
@@ -5830,7 +5830,7 @@ linux_process_target::async (bool enable)
        {
          delete_file_handler (linux_event_pipe.event_fd ());
 
-         linux_event_pipe.close ();
+         linux_event_pipe.close_pipe ();
        }
 
       gdb_sigmask (SIG_UNBLOCK, &mask, NULL);
index 2b56b2fac8ec92aad597f52c437157b5b497c2be..a1d34d59609a43830e1923168522f02ec1018206 100644 (file)
@@ -28,7 +28,7 @@
 event_pipe::~event_pipe ()
 {
   if (is_open ())
-    close ();
+    close_pipe ();
 }
 
 /* See event-pipe.h.  */
@@ -45,7 +45,7 @@ event_pipe::open ()
   if (fcntl (m_fds[0], F_SETFL, O_NONBLOCK) == -1
       || fcntl (m_fds[1], F_SETFL, O_NONBLOCK) == -1)
     {
-      close ();
+      close_pipe ();
       return false;
     }
 
@@ -55,7 +55,7 @@ event_pipe::open ()
 /* See event-pipe.h.  */
 
 void
-event_pipe::close ()
+event_pipe::close_pipe ()
 {
   ::close (m_fds[0]);
   ::close (m_fds[1]);
index 50679e470e40a599c49be6a2460c8b2955731823..9a41089774d6dbeb4cfe4f7160b4400ec4913d52 100644 (file)
@@ -34,10 +34,10 @@ public:
   DISABLE_COPY_AND_ASSIGN (event_pipe);
 
   /* Create a new pipe.  */
-  bool open ();
+  bool open_pipe ();
 
   /* Close the pipe.  */
-  void close ();
+  void close_pipe ();
 
   /* True if the event pipe has been opened.  */
   bool is_open () const