Check documentation of PyObject_CopyData #102383

Activity

1. terryjreedy commented on Mar 3, 2023

   

   terryjreedy

   on Mar 3, 2023

   Member

   More actions

   I few lines below, abstract.c has

       if (!PyObject_CheckBuffer(dest) ||
           !PyObject_CheckBuffer(src)) {
           PyErr_SetString(PyExc_TypeError,
                           "both destination and source must be "\
                           "bytes-like objects");
   

   So the 'PyObjects' indeed have to be buffer-like. I don't know the technical details, but I suspect that there is no real bug.
2. ronaldoussoren commented on Mar 3, 2023

   

   ronaldoussoren

   on Mar 3, 2023

   Contributor

   More actions

   I don't know the technical details, but I suspect that there is no real bug.

   There is a bug as the documentation does not match the implementation, Py_buffer is not a Python object. In this case IMHO the documentation should be fixed.
3. sobolevn commented on Mar 3, 2023

   

   sobolevn

   on Mar 3, 2023

   Member

   More actions

   Yes, PyObject_CheckBuffer looks like so:

   int
   PyObject_CheckBuffer(PyObject *obj)
   {
       PyBufferProcs *tp_as_buffer = Py_TYPE(obj)->tp_as_buffer;
       return (tp_as_buffer != NULL && tp_as_buffer->bf_getbuffer != NULL);
   }

   So, there's clearly a bug in the docs. PR is sent :)
   Thanks a lot for the report!
4. sobolevn

   added a commit that references this issue on Mar 3, 2023

   pythongh-102383: [docs] Arguments of `PyObject_CopyData` are `PyObject`s

   92835fe
5. sobolevn

   added

   type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

   docsDocumentation in the Doc dirDocumentation in the Doc dir

   on Mar 3, 2023
6. kumaraditya303

   added a commit that references this issue on Mar 3, 2023

   gh-102383: [docs] Arguments of `PyObject_CopyData` are `PyObject *` (#…

   ...

   7b91320
7. miss-islington

   added 2 commits that reference this issue on Mar 3, 2023

   pythongh-102383: [docs] Arguments of `PyObject_CopyData` are `PyObjec…

   ...

   b492daf

   gh-102383: [docs] Arguments of `PyObject_CopyData` are `PyObject *` (G…

   ...

   00791f2
8. carljm

   added a commit that references this issue on Mar 4, 2023

   Merge branch 'main' into nodeaddict

   ...

   a31af32
9. sobolevn

   closed this as completedon Mar 4, 2023