From 4ba009d8dc36c8dfdbdacfeaad75626565e33baa Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Tue, 31 May 2011 18:59:49 -0500 Subject: [PATCH] simply use the Python version for pyexpat.__version__ #12221 --- Misc/NEWS | 5 +++++ Modules/pyexpat.c | 26 +++++--------------------- 2 files changed, 10 insertions(+), 21 deletions(-) diff --git a/Misc/NEWS b/Misc/NEWS index f41f8959fac..2fc7b4765eb 100644 --- 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? =============================================== diff --git a/Modules/pyexpat.c b/Modules/pyexpat.c index 3bd68a9441b..3d04ca8c45d 100644 --- a/Modules/pyexpat.c +++ b/Modules/pyexpat.c @@ -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()); {