[binutils-gdb] Avoid conflict with gnulib open/close macros.

[Roland McGrath]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=8674f082e3f0d3f27ded5d93ebbd11cd702f5f04

commit 8674f082e3f0d3f27ded5d93ebbd11cd702f5f04
Author: Roland McGrath <mcgrathr@google.com>
Date:   Tue Mar 1 16:03:58 2022 -0800

    Avoid conflict with gnulib open/close macros.
    
    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.

Diff:
---
 gdb/inf-ptrace.h         | 4 ++--
 gdbserver/linux-low.cc   | 4 ++--
 gdbsupport/event-pipe.cc | 6 +++---
 gdbsupport/event-pipe.h  | 4 ++--
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/gdb/inf-ptrace.h b/gdb/inf-ptrace.h
index 62cc7778767..8f18d4579a6 100644
--- a/gdb/inf-ptrace.h
+++ b/gdb/inf-ptrace.h
@@ -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 ()
diff --git a/gdbserver/linux-low.cc b/gdbserver/linux-low.cc
index 301e42a36f3..0a5b6063104 100644
--- a/gdbserver/linux-low.cc
+++ b/gdbserver/linux-low.cc
@@ -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);
diff --git a/gdbsupport/event-pipe.cc b/gdbsupport/event-pipe.cc
index 2b56b2fac8e..a1d34d59609 100644
--- a/gdbsupport/event-pipe.cc
+++ b/gdbsupport/event-pipe.cc
@@ -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]);
diff --git a/gdbsupport/event-pipe.h b/gdbsupport/event-pipe.h
index 50679e470e4..9a41089774d 100644
--- a/gdbsupport/event-pipe.h
+++ b/gdbsupport/event-pipe.h
@@ -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