* pex-win32.c (fix_argv): Expand comment.

From-SVN: r82482

* pex-win32.c (fix_argv): Expand comment.
From-SVN: r82482
b47785f4 · Danny Smith · Danny Smith · 421563d5 · b47785f4 · b47785f4
Commit b47785f4 authored May 31, 2004 by Danny Smith Committed by Danny Smith May 31, 2004
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 1 deletions

libiberty/ChangeLog
+4 -0

libiberty/pex-win32.c
+6 -1

No files found.
--- a/libiberty/ChangeLog
+++ b/libiberty/ChangeLog
+2004-05-31  Danny Smith  <dannysmith@users.sourceforge.net>
+
+	* pex-win32.c (fix_argv): Expand comment.
+
 2004-05-25  Daniel Jacobowitz  <drow@false.org>

 	* Makefile.in: Add .NOEXPORT.

--- a/libiberty/pex-win32.c
+++ b/libiberty/pex-win32.c
@@ -59,7 +59,12 @@ fix_argv (argvec)
  int i;
  char * command0 = argvec[0];

-  /* Ensure that the executable pathname uses Win32 backslashes.  */
+  /* Ensure that the executable pathname uses Win32 backslashes. This
+     is not necessary on NT, but on W9x, forward slashes causes failure
+     of spawn* and exec* functions (and probably any function that
+     calls CreateProcess) *iff* the executable pathname (argvec[0]) is
+     a quoted string.  And quoting is necessary in case a pathname
+     contains  embedded white space. You can't win.  */
  for (; *command0 != '\0'; command0++)
    if (*command0 == '/')
      *command0 = '\\';