pitrou commented on PR #42034: URL: https://github.com/apache/arrow/pull/42034#issuecomment-2160856438
> I quickly tested its upgrade script on the pyarrow C++ code, and what it does change is replace a bunch of `.. == Py_None` cases with `Py_IsNone(..)`. > > But maybe the more interesting usage of the tool would be that we can use newer C API functions in our code that might make our PyArrow C++ easier to read or maintain? (but again not sure how much of those cases would actually apply here) Yes, the idea would be to use newer C API functions as in this PR. Vendoring it should be trivial: there is [a single `.h` file](https://github.com/python/pythoncapi-compat/blob/main/pythoncapi_compat.h). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
