Revert "use memset instead of a for-loop"

picnixz · picnixz · commit 39951be5d9c8 · 2025-01-22T14:44:12.000+01:00
This reverts commit d71b3a5.
diff --git a/Python/codecs.c b/Python/codecs.c
@@ -739,7 +739,9 @@ PyObject *PyCodec_ReplaceErrors(PyObject *exc)
         }
         assert(slen == 0 || PyUnicode_KIND(res) == PyUnicode_2BYTE_KIND);
         Py_UCS2 *outp = PyUnicode_2BYTE_DATA(res);
-        memset(outp, Py_UNICODE_REPLACEMENT_CHARACTER, sizeof(Py_UCS2) * slen);
+        for (Py_ssize_t i = 0; i < slen; ++i) {
+            outp[i] = Py_UNICODE_REPLACEMENT_CHARACTER;
+        }
         assert(_PyUnicode_CheckConsistency(res, 1));
         return Py_BuildValue("(Nn)", res, end);
     }

Original file line number	Diff line number	Diff line change
`@@ -739,7 +739,9 @@ PyObject PyCodec_ReplaceErrors(PyObject exc)`
`739`	`739`	`}`
`740`	`740`	`assert(slen == 0 \|\| PyUnicode_KIND(res) == PyUnicode_2BYTE_KIND);`
`741`	`741`	`Py_UCS2 *outp = PyUnicode_2BYTE_DATA(res);`
`742`		`- memset(outp, Py_UNICODE_REPLACEMENT_CHARACTER, sizeof(Py_UCS2) * slen);`
	`742`	`+ for (Py_ssize_t i = 0; i < slen; ++i) {`
	`743`	`+ outp[i] = Py_UNICODE_REPLACEMENT_CHARACTER;`
	`744`	`+ }`
`743`	`745`	`assert(_PyUnicode_CheckConsistency(res, 1));`
`744`	`746`	`return Py_BuildValue("(Nn)", res, end);`
`745`	`747`	`}`