Author: neal.norwitz
Date: Sun Aug 26 04:21:42 2007
New Revision: 57486

Modified:
   python/branches/py3k/Include/moduleobject.h
   python/branches/py3k/Objects/moduleobject.c
Log:
Get rid of more uses of string and use const in a few places.


Modified: python/branches/py3k/Include/moduleobject.h
==============================================================================
--- python/branches/py3k/Include/moduleobject.h (original)
+++ python/branches/py3k/Include/moduleobject.h Sun Aug 26 04:21:42 2007
@@ -14,8 +14,8 @@
 
 PyAPI_FUNC(PyObject *) PyModule_New(const char *);
 PyAPI_FUNC(PyObject *) PyModule_GetDict(PyObject *);
-PyAPI_FUNC(char *) PyModule_GetName(PyObject *);
-PyAPI_FUNC(char *) PyModule_GetFilename(PyObject *);
+PyAPI_FUNC(const char *) PyModule_GetName(PyObject *);
+PyAPI_FUNC(const char *) PyModule_GetFilename(PyObject *);
 PyAPI_FUNC(void) _PyModule_Clear(PyObject *);
 
 #ifdef __cplusplus

Modified: python/branches/py3k/Objects/moduleobject.c
==============================================================================
--- python/branches/py3k/Objects/moduleobject.c (original)
+++ python/branches/py3k/Objects/moduleobject.c Sun Aug 26 04:21:42 2007
@@ -54,7 +54,7 @@
        return d;
 }
 
-char *
+const char *
 PyModule_GetName(PyObject *m)
 {
        PyObject *d;
@@ -79,7 +79,7 @@
        return PyString_AsString(nameobj);
 }
 
-char *
+const char *
 PyModule_GetFilename(PyObject *m)
 {
        PyObject *d;
@@ -120,8 +120,8 @@
        /* First, clear only names starting with a single underscore */
        pos = 0;
        while (PyDict_Next(d, &pos, &key, &value)) {
-               if (value != Py_None && PyString_Check(key)) {
-                       char *s = PyString_AsString(key);
+               if (value != Py_None && PyUnicode_Check(key)) {
+                       const char *s = PyUnicode_AsString(key);
                        if (s[0] == '_' && s[1] != '_') {
                                if (Py_VerboseFlag > 1)
                                    PySys_WriteStderr("#   clear[1] %s\n", s);
@@ -133,8 +133,8 @@
        /* Next, clear all names except for __builtins__ */
        pos = 0;
        while (PyDict_Next(d, &pos, &key, &value)) {
-               if (value != Py_None && PyString_Check(key)) {
-                       char *s = PyString_AsString(key);
+               if (value != Py_None && PyUnicode_Check(key)) {
+                       const char *s = PyUnicode_AsString(key);
                        if (s[0] != '_' || strcmp(s, "__builtins__") != 0) {
                                if (Py_VerboseFlag > 1)
                                    PySys_WriteStderr("#   clear[2] %s\n", s);
@@ -187,8 +187,8 @@
 static PyObject *
 module_repr(PyModuleObject *m)
 {
-       char *name;
-       char *filename;
+       const char *name;
+       const char *filename;
 
        name = PyModule_GetName((PyObject *)m);
        if (name == NULL) {
_______________________________________________
Python-3000-checkins mailing list
[email protected]
http://mail.python.org/mailman/listinfo/python-3000-checkins

Reply via email to