- changed the nt.popen2 return values back to
(write, read, ...), based on feedback from GvR.
- added tuple-swapping code to popen2.py
- fixed some runaway indentation in posixmodule.c
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 8ea404e..2cfd39d 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -2295,9 +2295,9 @@
static int
_PyPopenCreateProcess(char *cmdstring,
- HANDLE hStdin,
- HANDLE hStdout,
- HANDLE hStderr)
+ HANDLE hStdin,
+ HANDLE hStdout,
+ HANDLE hStderr)
{
PROCESS_INFORMATION piProcInfo;
STARTUPINFO siStartInfo;
@@ -2360,15 +2360,15 @@
siStartInfo.wShowWindow = SW_HIDE;
if (CreateProcess(NULL,
- s2,
- NULL,
- NULL,
- TRUE,
- CREATE_NEW_CONSOLE,
- NULL,
- NULL,
- &siStartInfo,
- &piProcInfo) ) {
+ s2,
+ NULL,
+ NULL,
+ TRUE,
+ CREATE_NEW_CONSOLE,
+ NULL,
+ NULL,
+ &siStartInfo,
+ &piProcInfo) ) {
/* Close the handles now so anyone waiting is woken. */
CloseHandle(piProcInfo.hProcess);
CloseHandle(piProcInfo.hThread);
@@ -2404,9 +2404,9 @@
* the these handles; resulting in non-closeable handles to the pipes
* being created. */
fSuccess = DuplicateHandle(GetCurrentProcess(), hChildStdinWr,
- GetCurrentProcess(), &hChildStdinWrDup, 0,
- FALSE,
- DUPLICATE_SAME_ACCESS);
+ GetCurrentProcess(), &hChildStdinWrDup, 0,
+ FALSE,
+ DUPLICATE_SAME_ACCESS);
if (!fSuccess)
return win32_error("DuplicateHandle", NULL);
@@ -2418,9 +2418,8 @@
return win32_error("CreatePipe", NULL);
fSuccess = DuplicateHandle(GetCurrentProcess(), hChildStdoutRd,
- GetCurrentProcess(), &hChildStdoutRdDup, 0,
- FALSE,
- DUPLICATE_SAME_ACCESS);
+ GetCurrentProcess(), &hChildStdoutRdDup, 0,
+ FALSE, DUPLICATE_SAME_ACCESS);
if (!fSuccess)
return win32_error("DuplicateHandle", NULL);
@@ -2431,10 +2430,11 @@
if (n != POPEN_4) {
if (!CreatePipe(&hChildStderrRd, &hChildStderrWr, &saAttr, 0))
return win32_error("CreatePipe", NULL);
- fSuccess = DuplicateHandle(GetCurrentProcess(), hChildStderrRd,
- GetCurrentProcess(), &hChildStderrRdDup, 0,
- FALSE,
- DUPLICATE_SAME_ACCESS);
+ fSuccess = DuplicateHandle(GetCurrentProcess(),
+ hChildStderrRd,
+ GetCurrentProcess(),
+ &hChildStderrRdDup, 0,
+ FALSE, DUPLICATE_SAME_ACCESS);
if (!fSuccess)
return win32_error("DuplicateHandle", NULL);
/* Close the inheritable version of ChildStdErr that we're using. */
@@ -2516,7 +2516,7 @@
if (n != 4)
CloseHandle(hChildStderrRdDup);
- f = Py_BuildValue("OO",p2,p1);
+ f = Py_BuildValue("OO",p1,p2);
break;
}
@@ -2545,23 +2545,23 @@
PyFile_SetBufSize(p1, 0);
PyFile_SetBufSize(p2, 0);
PyFile_SetBufSize(p3, 0);
- f = Py_BuildValue("OOO",p2,p1,p3);
+ f = Py_BuildValue("OOO",p1,p2,p3);
break;
}
}
if (n == POPEN_4) {
if (!_PyPopenCreateProcess(cmdstring,
- hChildStdinRd,
- hChildStdoutWr,
- hChildStdoutWr))
+ hChildStdinRd,
+ hChildStdoutWr,
+ hChildStdoutWr))
return win32_error("CreateProcess", NULL);
}
else {
if (!_PyPopenCreateProcess(cmdstring,
- hChildStdinRd,
- hChildStdoutWr,
- hChildStderrWr))
+ hChildStdinRd,
+ hChildStdoutWr,
+ hChildStderrWr))
return win32_error("CreateProcess", NULL);
}