From: Barry Warsaw <barry@python.org>
Date: Tue, 9 Feb 1999 19:31:45 +0000 (+0000)
Subject: Got rid of the file-global PosixError.  This was redundant since it
X-Git-Tag: v1.5.2b2~95
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ca74da4e2c0ccbf7c8c54a7ed1a13385f0e205cb;p=python

Got rid of the file-global PosixError.  This was redundant since it
was just an alias for PyExc_OSError and the way we were doing it was
causing a (small) memory leak anyway.  Just use PyExc_OSError
everywhere.
---

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index f925d6e084..013e3b17d3 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -326,20 +326,18 @@ convertenviron()
 }
 
 
-static PyObject *PosixError; /* Exception posix.error */
-
 /* Set a POSIX-specific error from errno, and return NULL */
 
 static PyObject *
 posix_error()
 {
-	return PyErr_SetFromErrno(PosixError);
+	return PyErr_SetFromErrno(PyExc_OSError);
 }
 static PyObject *
 posix_error_with_filename(name)
 	char* name;
 {
-	return PyErr_SetFromErrnoWithFilename(PosixError, name);
+	return PyErr_SetFromErrnoWithFilename(PyExc_OSError, name);
 }
 
 
@@ -410,7 +408,7 @@ static PyObject * os2_error(int code)
 
     v = Py_BuildValue("(is)", code, text);
     if (v != NULL) {
-        PyErr_SetObject(PosixError, v);
+        PyErr_SetObject(PyExc_OSError, v);
         Py_DECREF(v);
     }
     return NULL; /* Signal to Python that an Exception is Pending */
@@ -3482,7 +3480,5 @@ INITFUNC()
         if (all_ins(d))
                 return;
 
-	Py_INCREF(PyExc_OSError);
-	PosixError = PyExc_OSError;
-	PyDict_SetItemString(d, "error", PosixError);
+	PyDict_SetItemString(d, "error", PyExc_OSError);
 }