upgrade_pythoncapi.py: Wrong transformations for Py_Is

Consider an example:

if (((PyObject*)obj)->data == Py_None) {
    111;
}
if (Py_None == ((PyObject*)obj)->data) {
    111;
}

After upgrade_pythoncapi.py I got:

#include "pythoncapi_compat.h"

if (((PyObject*)obj)->Py_IsNone(data)) {
    111;
}
if (Py_None == ((PyObject*)obj)->data) {
    111;
}

Real world example:
https://github.com/aleaxit/gmpy/blob/eb8dfcbd84abcfcb36b4adcb0d5c6d050731dd75/src/gmpy2_xmpz_misc.c#L237

I'm not sure if this is a bug. This kind of issues is not easy to avoid, using regexps for code transformations. Have you considered to use something like the pycparser?

Thanks for the project.

upgrade_pythoncapi.py から始めて、issue にある 2 つの C の例を再現し、その後、参照されている gmpy2_xmpz_misc.c のケースと出力を比較します。メンバーアクセスを壊さずに両方のオペランド順序を変換で処理できるか確認します。意図した Py_IsNone 変換を使って有効な C が生成されれば完了です。