]> granicus.if.org Git - python/commitdiff
simply use the Python version for pyexpat.__version__ #12221
authorBenjamin Peterson <benjamin@python.org>
Tue, 31 May 2011 23:59:49 +0000 (18:59 -0500)
committerBenjamin Peterson <benjamin@python.org>
Tue, 31 May 2011 23:59:49 +0000 (18:59 -0500)
Misc/NEWS
Modules/pyexpat.c

index f41f8959face392f4d079d0f38760c5bad80949a..2fc7b4765eb18920e7cc7f475a72b16b610b4b48 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -7,6 +7,11 @@ What's New in Python 2.7.2?
 
 *Release date: 2011-06-XX*
 
+Extension Modules
+-----------------
+
+- Issue #1221: Make pyexpat.__version__ equal to the Python version.
+
 
 What's New in Python 2.7.2 release candidate 1?
 ===============================================
index 3bd68a9441b6f4a6d8d62a16a40397bb0c4c190c..3d04ca8c45d0349ce8e690f4683af2a89343a3c7 100644 (file)
@@ -1792,26 +1792,6 @@ static struct PyMethodDef pyexpat_methods[] = {
 PyDoc_STRVAR(pyexpat_module_documentation,
 "Python wrapper for Expat parser.");
 
-/* Return a Python string that represents the version number without the
- * extra cruft added by revision control, even if the right options were
- * given to the "cvs export" command to make it not include the extra
- * cruft.
- */
-static PyObject *
-get_version_string(void)
-{
-    static char *rcsid = "$Revision$";
-    char *rev = rcsid;
-    int i = 0;
-
-    while (!isdigit(Py_CHARMASK(*rev)))
-        ++rev;
-    while (rev[i] != ' ' && rev[i] != '\0')
-        ++i;
-
-    return PyString_FromStringAndSize(rev, i);
-}
-
 /* Initialization function for the module */
 
 #ifndef MODULE_NAME
@@ -1841,6 +1821,7 @@ MODULE_INITFUNC(void)
     PyObject *modelmod_name;
     PyObject *model_module;
     PyObject *sys_modules;
+    PyObject *version;
     static struct PyExpat_CAPI capi;
     PyObject* capi_object;
 
@@ -1872,7 +1853,10 @@ MODULE_INITFUNC(void)
     Py_INCREF(&Xmlparsetype);
     PyModule_AddObject(m, "XMLParserType", (PyObject *) &Xmlparsetype);
 
-    PyModule_AddObject(m, "__version__", get_version_string());
+    version = PyString_FromString(PY_VERSION);
+    if (!version)
+        return;
+    PyModule_AddObject(m, "__version__", version);
     PyModule_AddStringConstant(m, "EXPAT_VERSION",
                                (char *) XML_ExpatVersion());
     {