JonathanSalwan / Triton

Triton is a dynamic binary analysis library. Build your own program analysis tools, automate your reverse engineering, perform software verification or just emulate code.
https://triton-library.github.io
Apache License 2.0
3.4k stars 524 forks source link

Including longintrepr.h is not necessary #1246

Closed hamarituc closed 1 year ago

hamarituc commented 1 year ago

During fixing a Python 3.11 build failure for Triton 0.9 (which was already fixed in the meantime) I discovered it is not necessary to include <longintrepr.h>. So it would be advisable to remove it as it is discouraged by upstream. See https://docs.python.org/3/whatsnew/3.11.html#whatsnew311-c-api-porting:

The non-limited API files cellobject.h, classobject.h, code.h, context.h, funcobject.h, genobject.h and longintrepr.h have been moved to the Include/cpython directory. Moreover, the eval.h header file was removed. These files must not be included directly, as they are already included in Python.h: Include Files. If they have been included directly, consider including Python.h instead. (Contributed by Victor Stinner in bpo-35134.)