diff options
author | Roland McGrath <mcgrathr@google.com> | 2022-03-01 16:03:58 -0800 |
---|---|---|
committer | Roland McGrath <mcgrathr@google.com> | 2022-03-03 11:21:36 -0800 |
commit | 8674f082e3f0d3f27ded5d93ebbd11cd702f5f04 (patch) | |
tree | 1e12f26e02b8e3909639f4ea7b4b675fb4ff5cfc | |
parent | db120fb808dc24538e89b851d6dda1890aad5a1f (diff) | |
download | gdb-8674f082e3f0d3f27ded5d93ebbd11cd702f5f04.zip gdb-8674f082e3f0d3f27ded5d93ebbd11cd702f5f04.tar.gz gdb-8674f082e3f0d3f27ded5d93ebbd11cd702f5f04.tar.bz2 |
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.
-rw-r--r-- | gdb/inf-ptrace.h | 4 | ||||
-rw-r--r-- | gdbserver/linux-low.cc | 4 | ||||
-rw-r--r-- | gdbsupport/event-pipe.cc | 6 | ||||
-rw-r--r-- | 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 62cc777..8f18d45 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 301e42a..0a5b606 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 2b56b2f..a1d34d5 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 50679e4..9a41089 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 |