Mercurial > cpython-lru_cache-weakref

diff Python/bltinmodule.c @ 27786:f34446fa216e legacy-trunk

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Change filterstring() and filterunicode(): If the object is not a real str or unicode but an instance of a subclass, construct the output via looping over __getitem__. This guarantees that the result is the same for function==None and function==lambda x:x This doesn't happen for tuples, because filtertuple() uses PyTuple_GetItem(). (This was discussed on SF bug #665835).
author Walter Dörwald <walter@livinglogic.de>
date Mon, 10 Feb 2003 13:19:13 +0000
parents 904f0e769820
children b650a54a76fd
line wrap: on
line diff
--- a/Python/bltinmodule.c	Mon Feb 10 13:08:04 2003 +0000
+++ b/Python/bltinmodule.c	Mon Feb 10 13:19:13 2003 +0000
@@ -1934,40 +1934,43 @@
 	int outlen = len;
 
 	if (func == Py_None) {
-		/* No character is ever false -- share input string
-		 * (if it's not a subclass) */
-		if (PyString_CheckExact(strobj))
+		/* If it's a real string we can return the original,
+		 * as no character is ever false and __getitem__
+		 * does return this character. If it's a subclass
+		 * we must go through the __getitem__ loop */
+		if (PyString_CheckExact(strobj)) {
 			Py_INCREF(strobj);
-		else
-			strobj = PyString_FromStringAndSize(
-				PyString_AS_STRING(strobj),
-				len
-			);
-		return strobj;
+			return strobj;
+		}
 	}
 	if ((result = PyString_FromStringAndSize(NULL, len)) == NULL)
 		return NULL;
 
 	for (i = j = 0; i < len; ++i) {
-		PyObject *item, *arg, *good;
+		PyObject *item;
 		int ok;
 
 		item = (*strobj->ob_type->tp_as_sequence->sq_item)(strobj, i);
 		if (item == NULL)
 			goto Fail_1;
-		arg = Py_BuildValue("(O)", item);
-		if (arg == NULL) {
-			Py_DECREF(item);
-			goto Fail_1;
+		if (func==Py_None) {
+			ok = 1;
+		} else {
+			PyObject *arg, *good;
+			arg = Py_BuildValue("(O)", item);
+			if (arg == NULL) {
+				Py_DECREF(item);
+				goto Fail_1;
+			}
+			good = PyEval_CallObject(func, arg);
+			Py_DECREF(arg);
+			if (good == NULL) {
+				Py_DECREF(item);
+				goto Fail_1;
+			}
+			ok = PyObject_IsTrue(good);
+			Py_DECREF(good);
 		}
-		good = PyEval_CallObject(func, arg);
-		Py_DECREF(arg);
-		if (good == NULL) {
-			Py_DECREF(item);
-			goto Fail_1;
-		}
-		ok = PyObject_IsTrue(good);
-		Py_DECREF(good);
 		if (ok) {
 			int reslen;
 			if (!PyString_Check(item)) {
@@ -2026,16 +2029,14 @@
 	int outlen = len;
 
 	if (func == Py_None) {
-		/* No character is ever false -- share input string
-		 * (it if's not a subclass) */
-		if (PyUnicode_CheckExact(strobj))
+		/* If it's a real string we can return the original,
+		 * as no character is ever false and __getitem__
+		 * does return this character. If it's a subclass
+		 * we must go through the __getitem__ loop */
+		if (PyUnicode_CheckExact(strobj)) {
 			Py_INCREF(strobj);
-		else
-			strobj = PyUnicode_FromUnicode(
-				PyUnicode_AS_UNICODE(strobj),
-				len
-			);
-		return strobj;
+			return strobj;
+		}
 	}
 	if ((result = PyUnicode_FromUnicode(NULL, len)) == NULL)
 		return NULL;
@@ -2047,19 +2048,23 @@
 		item = (*strobj->ob_type->tp_as_sequence->sq_item)(strobj, i);
 		if (item == NULL)
 			goto Fail_1;
-		arg = Py_BuildValue("(O)", item);
-		if (arg == NULL) {
-			Py_DECREF(item);
-			goto Fail_1;
+		if (func == Py_None) {
+			ok = 1;
+		} else {
+			arg = Py_BuildValue("(O)", item);
+			if (arg == NULL) {
+				Py_DECREF(item);
+				goto Fail_1;
+			}
+			good = PyEval_CallObject(func, arg);
+			Py_DECREF(arg);
+			if (good == NULL) {
+				Py_DECREF(item);
+				goto Fail_1;
+			}
+			ok = PyObject_IsTrue(good);
+			Py_DECREF(good);
 		}
-		good = PyEval_CallObject(func, arg);
-		Py_DECREF(arg);
-		if (good == NULL) {
-			Py_DECREF(item);
-			goto Fail_1;
-		}
-		ok = PyObject_IsTrue(good);
-		Py_DECREF(good);
 		if (ok) {
 			int reslen;
 			if (!PyUnicode_Check(item)) {