Dec 15, 2025 · Dec 15, 2025 · Dec 15, 2025 · Dec 15, 2025 · Dec 16, 2025 · Dec 16, 2025
diff --git a/Modules/xxlimited.c b/Modules/xxlimited.c
 // This is the implementation of Xxo.__new__; it takes arbitrary positional
 // and keyword arguments.
 static PyObject *
 Xxo_new(PyTypeObject *type, PyObject *args, PyObject *kwargs) {
 Xxo_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)
 {
    // Validate that we did not get any arguments.
    if ((args != NULL && PyObject_Length(args))
        || (kwargs != NULL && PyObject_Length(kwargs)))
Original file line number	Diff line number	Diff line change
Expand Up		@@ -160,7 +160,8 @@ Xxo_get_data(PyObject *self)
		// This is the implementation of Xxo.__new__; it takes arbitrary positional
		// and keyword arguments.
Copy link Member vstinnerDec 17, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I don't understand "it takes arbitrary positional and keyword arguments" since the error message below is "`Xxo.__new__()` takes no arguments". encukou reacted with thumbs up emoji Copy link MemberAuthor encukouDec 17, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. The`tp_new` calling convention takes args&kwargs, so we need to check. I agree the comment was misleading. Also the situation should be clear without the comment, so I removed it.
		static PyObject *
		Xxo_new(PyTypeObject type, PyObject args, PyObject *kwargs) {
		Xxo_new(PyTypeObject type, PyObject args, PyObject *kwargs)
		{
		// Validate that we did not get any arguments.
		if ((args != NULL && PyObject_Length(args))
		\|\| (kwargs != NULL && PyObject_Length(kwargs)))
Expand Down