Remove support for __members__ and __methods__. There still might be

nnorwitz · nnorwitz · commit 8dfc4a9baca7 · 2007-08-11T06:39:53.000Z
some cleanup to do on this.  Particularly in Python/traceback.c with
getting rid of the getattr if possible and Demo/*metaclasses/Enum.py.
diff --git a/Doc/lib/libstdtypes.tex b/Doc/lib/libstdtypes.tex
@@ -2051,16 +2051,6 @@ \section{Special Attributes \label{specialattrs}}
 object's (writable) attributes.
 \end{memberdesc}
 
-\begin{memberdesc}[object]{__methods__}
-\deprecated{2.2}{Use the built-in function \function{dir()} to get a
-list of an object's attributes.  This attribute is no longer available.}
-\end{memberdesc}
-
-\begin{memberdesc}[object]{__members__}
-\deprecated{2.2}{Use the built-in function \function{dir()} to get a
-list of an object's attributes.  This attribute is no longer available.}
-\end{memberdesc}
-
 \begin{memberdesc}[instance]{__class__}
 The class to which a class instance belongs.
 \end{memberdesc}
diff --git a/Include/structmember.h b/Include/structmember.h
@@ -19,20 +19,12 @@ extern "C" {
 #define offsetof(type, member) ( (int) & ((type*)0) -> member )
 #endif
 
-/* An array of memberlist structures defines the name, type and offset
+/* An array of PyMemberDef structures defines the name, type and offset
    of selected members of a C structure.  These can be read by
-   PyMember_Get() and set by PyMember_Set() (except if their READONLY flag
-   is set).  The array must be terminated with an entry whose name
+   PyMember_GetOne() and set by PyMember_SetOne() (except if their READONLY
+   flag is set).  The array must be terminated with an entry whose name
    pointer is NULL. */
 
-struct memberlist {
-	/* Obsolete version, for binary backwards compatibility */
-	char *name;
-	int type;
-	int offset;
-	int flags;
-};
-
 typedef struct PyMemberDef {
 	/* Current version, use this */
 	char *name;
@@ -76,16 +68,11 @@ typedef struct PyMemberDef {
 
 /* Flags */
 #define READONLY	1
-#define RO		READONLY		/* Shorthand */
 #define READ_RESTRICTED	2
 #define WRITE_RESTRICTED 4
 #define RESTRICTED	(READ_RESTRICTED | WRITE_RESTRICTED)
 
 
-/* Obsolete API, for binary backwards compatibility */
-PyAPI_FUNC(PyObject *) PyMember_Get(const char *, struct memberlist *, const char *);
-PyAPI_FUNC(int) PyMember_Set(char *, struct memberlist *, const char *, PyObject *);
-
 /* Current API, use this */
 PyAPI_FUNC(PyObject *) PyMember_GetOne(const char *, struct PyMemberDef *);
 PyAPI_FUNC(int) PyMember_SetOne(char *, struct PyMemberDef *, PyObject *);
diff --git a/Lib/test/test_descrtut.py b/Lib/test/test_descrtut.py
@@ -162,15 +162,7 @@ def merge(self, other):
     True
     >>>
 
-Under the new proposal, the __methods__ attribute no longer exists:
-
-    >>> [].__methods__
-    Traceback (most recent call last):
-      File "<stdin>", line 1, in ?
-    AttributeError: 'list' object has no attribute '__methods__'
-    >>>
-
-Instead, you can get the same information from the list type:
+You can get the information from the list type:
 
     >>> pprint.pprint(dir(list))    # like list.__dict__.keys(), but sorted
     ['__add__',
diff --git a/Lib/test/test_traceback.py b/Lib/test/test_traceback.py
@@ -52,14 +52,6 @@ def test_bad_indentation(self):
         self.assert_("^" in err[2])
         self.assertEqual(err[1].find(")"), err[2].find("^"))
 
-    def test_members(self):
-        # Covers Python/structmember.c::listmembers()
-        try:
-            1/0
-        except:
-            import sys
-            sys.exc_info()[2].__members__
-
     def test_base_exception(self):
         # Test that exceptions derived from BaseException are formatted right
         e = KeyboardInterrupt()
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -75,6 +75,8 @@ Core and Builtins
   its argument, if it exists. If not, it will work like before. This allows
   customizing the output of dir() in the presence of a __getattr__().
 
+- Removed support for __members__ and __methods__.
+
 - Removed indexing/slicing on BaseException.
 
 - input() became raw_input(): the name input() now implements the
@@ -219,7 +221,7 @@ C API
   __coerce__, __div__, __idiv__, __rdiv__
 
 - Removed these C APIs:
-  PyNumber_Coerce(), PyNumber_CoerceEx() 
+  PyNumber_Coerce(), PyNumber_CoerceEx(), PyMember_Get, PyMember_Set
 
 - Removed these C slots/fields:
   nb_divide, nb_inplace_divide
@@ -229,7 +231,8 @@ C API
 
 - Removed these typedefs:
   intargfunc, intintargfunc, intobjargproc, intintobjargproc,
-  getreadbufferproc, getwritebufferproc, getsegcountproc, getcharbufferproc
+  getreadbufferproc, getwritebufferproc, getsegcountproc, getcharbufferproc,
+  memberlist
 
 
 Tests
diff --git a/Misc/find_recursionlimit.py b/Misc/find_recursionlimit.py
@@ -17,8 +17,6 @@
 It ends when Python causes a segmentation fault because the limit is
 too high.  On platforms like Mac and Windows, it should exit with a
 MemoryError.
-
-NB: A program that does not use __methods__ can set a higher limit.
 """
 
 import sys
diff --git a/Modules/pyexpat.c b/Modules/pyexpat.c
@@ -1400,6 +1400,12 @@ xmlparse_getattr(xmlparseobject *self, char *name)
         }
     }
 
+    return Py_FindMethod(xmlparse_methods, (PyObject *)self, name);
+}
+
+static PyObject *
+xmlparse_dir(PyObject *self, PyObject* noargs)
+{
 #define APPEND(list, str)				\
         do {						\
                 PyObject *o = PyString_FromString(str);	\
@@ -1408,36 +1414,39 @@ xmlparse_getattr(xmlparseobject *self, char *name)
                 Py_XDECREF(o);				\
         } while (0)
 
-    if (strcmp(name, "__members__") == 0) {
-        int i;
-        PyObject *rc = PyList_New(0);
-	if (!rc)
-		return NULL;
-        for (i = 0; handler_info[i].name != NULL; i++) {
-            PyObject *o = get_handler_name(&handler_info[i]);
-            if (o != NULL)
-                PyList_Append(rc, o);
-            Py_XDECREF(o);
-        }
-        APPEND(rc, "ErrorCode");
-        APPEND(rc, "ErrorLineNumber");
-        APPEND(rc, "ErrorColumnNumber");
-        APPEND(rc, "ErrorByteIndex");
-        APPEND(rc, "CurrentLineNumber");
-        APPEND(rc, "CurrentColumnNumber");
-        APPEND(rc, "CurrentByteIndex");
-        APPEND(rc, "buffer_size");
-        APPEND(rc, "buffer_text");
-        APPEND(rc, "buffer_used");
-        APPEND(rc, "namespace_prefixes");
-        APPEND(rc, "ordered_attributes");
-        APPEND(rc, "specified_attributes");
-        APPEND(rc, "intern");
+    int i;
+    PyObject *rc = PyList_New(0);
+    if (!rc)
+	return NULL;
+    for (i = 0; handler_info[i].name != NULL; i++) {
+        PyObject *o = get_handler_name(&handler_info[i]);
+        if (o != NULL)
+            PyList_Append(rc, o);
+        Py_XDECREF(o);
+    }
+    APPEND(rc, "ErrorCode");
+    APPEND(rc, "ErrorLineNumber");
+    APPEND(rc, "ErrorColumnNumber");
+    APPEND(rc, "ErrorByteIndex");
+    APPEND(rc, "CurrentLineNumber");
+    APPEND(rc, "CurrentColumnNumber");
+    APPEND(rc, "CurrentByteIndex");
+    APPEND(rc, "buffer_size");
+    APPEND(rc, "buffer_text");
+    APPEND(rc, "buffer_used");
+    APPEND(rc, "namespace_prefixes");
+    APPEND(rc, "ordered_attributes");
+    APPEND(rc, "specified_attributes");
+    APPEND(rc, "intern");
 
 #undef APPEND
-        return rc;
+
+    if (PyErr_Occurred()) {
+        Py_DECREF(rc);
+        rc = NULL;
     }
-    return Py_FindMethod(xmlparse_methods, (PyObject *)self, name);
+
+    return rc;
 }
 
 static int
@@ -1560,6 +1569,10 @@ xmlparse_clear(xmlparseobject *op)
 
 PyDoc_STRVAR(Xmlparsetype__doc__, "XML parser");
 
+static PyMethodDef xmlparse_tp_methods[] = {
+    {"__dir__", xmlparse_dir, METH_NOARGS}
+};
+
 static PyTypeObject Xmlparsetype = {
 	PyVarObject_HEAD_INIT(NULL, 0)
 	"pyexpat.xmlparser",		/*tp_name*/
@@ -1588,7 +1601,12 @@ static PyTypeObject Xmlparsetype = {
 #endif
 	Xmlparsetype__doc__, /* tp_doc - Documentation string */
 	(traverseproc)xmlparse_traverse,	/* tp_traverse */
-	(inquiry)xmlparse_clear		/* tp_clear */
+	(inquiry)xmlparse_clear,		/* tp_clear */
+	0,				/* tp_richcompare */
+	0,				/* tp_weaklistoffset */
+	0,				/* tp_iter */
+	0,				/* tp_iternext */
+	xmlparse_tp_methods		/* tp_methods */
 };
 
 /* End of code for xmlparser objects */
diff --git a/Objects/methodobject.c b/Objects/methodobject.c
@@ -275,47 +275,12 @@ PyTypeObject PyCFunction_Type = {
 	0,					/* tp_dict */
 };
 
-/* List all methods in a chain -- helper for findmethodinchain */
-
-static PyObject *
-listmethodchain(PyMethodChain *chain)
-{
-	PyMethodChain *c;
-	PyMethodDef *ml;
-	int i, n;
-	PyObject *v;
-
-	n = 0;
-	for (c = chain; c != NULL; c = c->link) {
-		for (ml = c->methods; ml->ml_name != NULL; ml++)
-			n++;
-	}
-	v = PyList_New(n);
-	if (v == NULL)
-		return NULL;
-	i = 0;
-	for (c = chain; c != NULL; c = c->link) {
-		for (ml = c->methods; ml->ml_name != NULL; ml++) {
-			PyList_SetItem(v, i, PyUnicode_FromString(ml->ml_name));
-			i++;
-		}
-	}
-	if (PyErr_Occurred()) {
-		Py_DECREF(v);
-		return NULL;
-	}
-	PyList_Sort(v);
-	return v;
-}
-
 /* Find a method in a method chain */
 
 PyObject *
 Py_FindMethodInChain(PyMethodChain *chain, PyObject *self, const char *name)
 {
 	if (name[0] == '_' && name[1] == '_') {
-		if (strcmp(name, "__methods__") == 0)
-			return listmethodchain(chain);
 		if (strcmp(name, "__doc__") == 0) {
 			const char *doc = self->ob_type->tp_doc;
 			if (doc != NULL)
diff --git a/PC/_winreg.c b/PC/_winreg.c
@@ -473,8 +473,8 @@ PyTypeObject PyHKEY_Type =
 
 #define OFF(e) offsetof(PyHKEYObject, e)
 
-static struct memberlist PyHKEY_memberlist[] = {
-	{"handle",      T_INT,      OFF(hkey)},
+static PyMemberDef PyHKEY_memberlist[] = {
+	{"handle",      T_INT,      OFF(hkey), READONLY},
 	{NULL}    /* Sentinel */
 };
 
@@ -523,7 +523,10 @@ PyHKEY_getattr(PyObject *self, const char *name)
 	PyErr_Clear();
 	if (strcmp(name, "handle") == 0)
 		return PyLong_FromVoidPtr(((PyHKEYObject *)self)->hkey);
-	return PyMember_Get((char *)self, PyHKEY_memberlist, name);
+	PyErr_Format(PyExc_AttributeError,
+                     "'%.50s' object has no attribute '%.400s'",
+                     Py_Type(self)->tp_name, name);
+	return NULL;
 }
 
 /************************************************************************
diff --git a/PC/os2emx/python25.def b/PC/os2emx/python25.def
@@ -1091,10 +1091,8 @@ EXPORTS
   "_Py_QnewFlag"
 
 ; From python25_s.lib(structmember)
-  "PyMember_Get"
   "PyMember_GetOne"
   "PyMember_SetOne"
-  "PyMember_Set"
 
 ; From python25_s.lib(symtable)
   "PySymtable_Build"
diff --git a/PC/os2vacpp/python.def b/PC/os2vacpp/python.def
@@ -244,8 +244,6 @@ EXPORTS
                PyMem_Free
                PyMem_Malloc
                PyMem_Realloc
-               PyMember_Get
-               PyMember_Set
                PyMethod_Class
                PyMethod_Fini
                PyMethod_Function
diff --git a/Python/structmember.c b/Python/structmember.c
diff --git a/Python/traceback.c b/Python/traceback.c
diff --git a/Tools/modulator/Templates/object_structure b/Tools/modulator/Templates/object_structure
