2005-05-11 Eli Zaretskii <eliz@gnu.org>

* pex-djgpp.c: Include string.h, fcntl.h, unistd.h, and
	sys/stat.h.
	(pex_init): Fix last argument to pex_init_common.
	(pex_djgpp_exec_child): Remove leading underscore from _open,
	_dup, _dup2, _close, and _spawnv/_spawnvp.  Replace `program',
	which is undeclared, with `executable', which was unused.  Remove
	unused variable `e'.  Fix casting of last arg to spawnv/spawnvp.
	(pex_djgpp_wait): Declare arguments with ATTRIBUTE_UNUSED.
This commit is contained in:
Ian Lance Taylor 2005-05-12 03:45:53 +00:00
parent be2c2e39a9
commit 282d9ec36b
2 changed files with 52 additions and 38 deletions

View file

@ -1,3 +1,14 @@
2005-05-11 Eli Zaretskii <eliz@gnu.org>
* pex-djgpp.c: Include string.h, fcntl.h, unistd.h, and
sys/stat.h.
(pex_init): Fix last argument to pex_init_common.
(pex_djgpp_exec_child): Remove leading underscore from _open,
_dup, _dup2, _close, and _spawnv/_spawnvp. Replace `program',
which is undeclared, with `executable', which was unused. Remove
unused variable `e'. Fix casting of last arg to spawnv/spawnvp.
(pex_djgpp_wait): Declare arguments with ATTRIBUTE_UNUSED.
2005-05-11 Paul Brook <paul@codesourcery.com> 2005-05-11 Paul Brook <paul@codesourcery.com>
* Makefile.in: Regenerate dependencies. * Makefile.in: Regenerate dependencies.

View file

@ -29,6 +29,10 @@ extern int errno;
#ifdef HAVE_STDLIB_H #ifdef HAVE_STDLIB_H
#include <stdlib.h> #include <stdlib.h>
#endif #endif
#include <string.h>
#include <fcntl.h>
#include <unistd.h>
#include <sys/stat.h>
#include <process.h> #include <process.h>
/* Use ECHILD if available, otherwise use EINVAL. */ /* Use ECHILD if available, otherwise use EINVAL. */
@ -68,7 +72,7 @@ pex_init (int flags, const char *pname, const char *tempbase)
{ {
/* DJGPP does not support pipes. */ /* DJGPP does not support pipes. */
flags &= ~ PEX_USE_PIPES; flags &= ~ PEX_USE_PIPES;
return pex_init_common (flags, pname, tempbase, funcs); return pex_init_common (flags, pname, tempbase, &funcs);
} }
/* Open a file for reading. */ /* Open a file for reading. */
@ -119,46 +123,46 @@ pex_djgpp_exec_child (struct pex_obj *obj, int flags, const char *executable,
if (in != STDIN_FILE_NO) if (in != STDIN_FILE_NO)
{ {
org_in = _dup (STDIN_FILE_NO); org_in = dup (STDIN_FILE_NO);
if (org_in < 0) if (org_in < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup"; *errmsg = "dup";
return -1; return -1;
} }
if (_dup2 (in, STDIN_FILE_NO) < 0) if (dup2 (in, STDIN_FILE_NO) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup2"; *errmsg = "dup2";
return -1; return -1;
} }
if (_close (in) < 0) if (close (in) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_close"; *errmsg = "close";
return -1; return -1;
} }
} }
if (out != STDOUT_FILE_NO) if (out != STDOUT_FILE_NO)
{ {
org_out = _dup (STDOUT_FILE_NO); org_out = dup (STDOUT_FILE_NO);
if (org_out < 0) if (org_out < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup"; *errmsg = "dup";
return -1; return -1;
} }
if (_dup2 (out, STDOUT_FILE_NO) < 0) if (dup2 (out, STDOUT_FILE_NO) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup2"; *errmsg = "dup2";
return -1; return -1;
} }
if (_close (out) < 0) if (close (out) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_close"; *errmsg = "close";
return -1; return -1;
} }
} }
@ -166,70 +170,68 @@ pex_djgpp_exec_child (struct pex_obj *obj, int flags, const char *executable,
if (errdes != STDERR_FILE_NO if (errdes != STDERR_FILE_NO
|| (flags & PEX_STDERR_TO_STDOUT) != 0) || (flags & PEX_STDERR_TO_STDOUT) != 0)
{ {
int e; org_errdes = dup (STDERR_FILE_NO);
org_errdes = _dup (STDERR_FILE_NO);
if (org_errdes < 0) if (org_errdes < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup"; *errmsg = "dup";
return -1; return -1;
} }
if (_dup2 ((flags & PEX_STDERR_TO_STDOUT) != 0 ? STDOUT_FILE_NO : errdes, if (dup2 ((flags & PEX_STDERR_TO_STDOUT) != 0 ? STDOUT_FILE_NO : errdes,
STDERR_FILE_NO) < 0) STDERR_FILE_NO) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup2"; *errmsg = "dup2";
return -1; return -1;
} }
if (errdes != STDERR_FILE_NO) if (errdes != STDERR_FILE_NO)
{ {
if (_close (errdes) < 0) if (close (errdes) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_close"; *errmsg = "close";
return -1; return -1;
} }
} }
} }
status = (((flags & PEX_SEARCH) != 0 ? _spawnvp : _spawnv) status = (((flags & PEX_SEARCH) != 0 ? spawnvp : spawnv)
(P_WAIT, program, (const char **) argv)); (P_WAIT, executable, (char * const *) argv));
if (status == -1) if (status == -1)
{ {
*err = errno; *err = errno;
*errmsg = ((flags & PEX_SEARCH) != 0) ? "_spawnvp" : "_spawnv"; *errmsg = ((flags & PEX_SEARCH) != 0) ? "spawnvp" : "spawnv";
} }
if (in != STDIN_FILE_NO) if (in != STDIN_FILE_NO)
{ {
if (_dup2 (org_in, STDIN_FILE_NO) < 0) if (dup2 (org_in, STDIN_FILE_NO) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup2"; *errmsg = "dup2";
return -1; return -1;
} }
if (_close (org_in) < 0) if (close (org_in) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_close"; *errmsg = "close";
return -1; return -1;
} }
} }
if (out != STDOUT_FILE_NO) if (out != STDOUT_FILE_NO)
{ {
if (_dup2 (org_out, STDOUT_FILE_NO) < 0) if (dup2 (org_out, STDOUT_FILE_NO) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup2"; *errmsg = "dup2";
return -1; return -1;
} }
if (_close (org_out) < 0) if (close (org_out) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_close"; *errmsg = "close";
return -1; return -1;
} }
} }
@ -237,16 +239,16 @@ pex_djgpp_exec_child (struct pex_obj *obj, int flags, const char *executable,
if (errdes != STDERR_FILE_NO if (errdes != STDERR_FILE_NO
|| (flags & PEX_STDERR_TO_STDOUT) != 0) || (flags & PEX_STDERR_TO_STDOUT) != 0)
{ {
if (_dup2 (org_errdes, STDERR_FILE_NO) < 0) if (dup2 (org_errdes, STDERR_FILE_NO) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_dup2"; *errmsg = "dup2";
return -1; return -1;
} }
if (_close (org_errdes) < 0) if (close (org_errdes) < 0)
{ {
*err = errno; *err = errno;
*errmsg = "_close"; *errmsg = "close";
return -1; return -1;
} }
} }
@ -268,8 +270,9 @@ pex_djgpp_exec_child (struct pex_obj *obj, int flags, const char *executable,
static int static int
pex_djgpp_wait (struct pex_obj *obj, long pid, int *status, pex_djgpp_wait (struct pex_obj *obj, long pid, int *status,
struct pex_time *time, int done, const char **errmsg, struct pex_time *time, int done ATTRIBUTE_UNUSED,
int *err) const char **errmsg ATTRIBUTE_UNUSED,
int *err ATTRIBUTE_UNUSED)
{ {
int *statuses; int *statuses;