diff options
author | Greg Ward <gward@python.net> | 2000-03-01 21:51:56 (GMT) |
---|---|---|
committer | Greg Ward <gward@python.net> | 2000-03-01 21:51:56 (GMT) |
commit | b48bc17d10fa53c7ccbcca5597ee4bdb8754973e (patch) | |
tree | 355410253d73c75176611a0026e6fc010328832e | |
parent | 9217fcbb38887f2d0a25fb798d4df5fa9edcd797 (diff) | |
download | cpython-b48bc17d10fa53c7ccbcca5597ee4bdb8754973e.zip cpython-b48bc17d10fa53c7ccbcca5597ee4bdb8754973e.tar.gz cpython-b48bc17d10fa53c7ccbcca5597ee4bdb8754973e.tar.bz2 |
Second attempt to fix the ctermid_r/tmpnam_r warnings: define USE_CTERMID_R
and USE_TMPNAM_R at the top of the file and refer to them later; this
catches a second reference to 'tmpnam_r' that I didn't spot first time around.
-rw-r--r-- | Modules/posixmodule.c | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c index 4e50f1b..8ae5d2f 100644 --- a/Modules/posixmodule.c +++ b/Modules/posixmodule.c @@ -274,6 +274,16 @@ extern int lstat Py_PROTO((const char *, struct stat *)); #endif /* UNION_WAIT */ +/* Don't use the "_r" form if we don't need it (also, won't have a + prototype for it, at least on Solaris -- maybe others as well?). */ +#if defined(HAVE_CTERMID_R) && defined(WITH_THREAD) +#define USE_CTERMID_R +#endif + +#if defined(HAVE_TMPNAM_R) && defined(WITH_THREAD) +#define USE_TMPNAM_R +#endif + /* Return a dictionary corresponding to the POSIX environment table */ #if !defined(_MSC_VER) && ( !defined(__WATCOMC__) || defined(__QNX__) ) @@ -649,7 +659,7 @@ posix_ctermid(self, args) if (!PyArg_ParseTuple(args, ":ctermid")) return NULL; -#if defined(HAVE_CTERMID_R) && defined(WITH_THREAD) +#ifdef USE_CTERMID_R ret = ctermid_r(buffer); #else ret = ctermid(buffer); @@ -3342,7 +3352,7 @@ posix_tmpnam(self, args) if (!PyArg_ParseTuple(args, ":tmpnam")) return NULL; -#if defined(HAVE_TMPNAM_R) && defined(WITH_THREAD) +#ifdef USE_TMPNAM_R name = tmpnam_r(buffer); #else name = tmpnam(buffer); @@ -3350,7 +3360,7 @@ posix_tmpnam(self, args) if (name == NULL) { PyErr_SetObject(PyExc_OSError, Py_BuildValue("is", 0, -#ifdef HAVE_TMPNAM_R +#ifdef USE_TMPNAM_R "unexpected NULL from tmpnam_r" #else "unexpected NULL from tmpnam" |