Fix Cygwin compilation after target_ops C++ conversion.

Message ID 20180714125203.44168-1-jon.turney@dronecode.org.uk
State New, archived
Headers

Commit Message

Jon Turney July 14, 2018, 12:52 p.m. UTC
  After f6ac5f3d "Convert struct target_ops to C++", we need to explicitly use
the global namespace when calling ::close() from windows_nat_target methods,
as that object has a close() method.

gdb/ChangeLog:

2018-07-14  Jon Turney  <jon.turney@dronecode.org.uk>

	* windows-nat.c (windows_nat_target::create_inferior): Update to
	call close() in global namespace.
---
 gdb/ChangeLog     | 5 +++++
 gdb/windows-nat.c | 8 ++++----
 2 files changed, 9 insertions(+), 4 deletions(-)
  

Comments

Tom Tromey July 25, 2018, 8:08 p.m. UTC | #1
>>>>> "Jon" == Jon Turney <jon.turney@dronecode.org.uk> writes:

Jon> After f6ac5f3d "Convert struct target_ops to C++", we need to explicitly use
Jon> the global namespace when calling ::close() from windows_nat_target methods,
Jon> as that object has a close() method.

I didn't see a reply to this.

Jon> 2018-07-14  Jon Turney  <jon.turney@dronecode.org.uk>

Jon> 	* windows-nat.c (windows_nat_target::create_inferior): Update to
Jon> 	call close() in global namespace.

This is ok.  Thank you.

Tom
  

Patch

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 7b808eb3a7..74d4fcfdbe 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@ 
+2018-07-14  Jon Turney  <jon.turney@dronecode.org.uk>
+
+	* windows-nat.c (windows_nat_target::create_inferior): Update to
+	call close() in global namespace.
+
 2018-07-13  Andreas Arnez  <arnez@linux.vnet.ibm.com>
 
 	* s390-tdep.c (s390_displaced_step_fixup): Adjust PC for a
diff --git a/gdb/windows-nat.c b/gdb/windows-nat.c
index b5c00d7793..070aabe0de 100644
--- a/gdb/windows-nat.c
+++ b/gdb/windows-nat.c
@@ -2701,13 +2701,13 @@  windows_nat_target::create_inferior (const char *exec_file,
 
   if (tty >= 0)
     {
-      close (tty);
+      ::close (tty);
       dup2 (ostdin, 0);
       dup2 (ostdout, 1);
       dup2 (ostderr, 2);
-      close (ostdin);
-      close (ostdout);
-      close (ostderr);
+      ::close (ostdin);
+      ::close (ostdout);
+      ::close (ostderr);
     }
 #else  /* !__CYGWIN__ */
   allargs_len = strlen (allargs);