diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2021-12-23 20:20:46 -0500 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@efficios.com> | 2022-03-23 07:41:19 -0400 |
| commit | edae3fd6600f10f9e16dc017b705959f541ed19a (patch) | |
| tree | 0e8b6288c02dec48f369594a9032dcac20146e33 /gdb/python/py-membuf.c | |
| parent | e52a16f2aa20773f42c28bf91a568d0683e5767c (diff) | |
| download | binutils-edae3fd6600f10f9e16dc017b705959f541ed19a.zip binutils-edae3fd6600f10f9e16dc017b705959f541ed19a.tar.gz binutils-edae3fd6600f10f9e16dc017b705959f541ed19a.tar.bz2 | |
gdb/python: remove Python 2 support
New in this version:
- Add a PY_MAJOR_VERSION check in configure.ac / AC_TRY_LIBPYTHON. If
the user passes --with-python=python2, this will cause a configure
failure saying that GDB only supports Python 3.
Support for Python 2 is a maintenance burden for any patches touching
Python support. Among others, the differences between Python 2 and 3
string and integer types are subtle. It requires a lot of effort and
thinking to get something that behaves correctly on both. And that's if
the author and reviewer of the patch even remember to test with Python
2.
See this thread for an example:
https://sourceware.org/pipermail/gdb-patches/2021-December/184260.html
So, remove Python 2 support. Update the documentation to state that GDB
can be built against Python 3 (as opposed to Python 2 or 3).
Update all the spots that use:
- sys.version_info
- IS_PY3K
- PY_MAJOR_VERSION
- gdb_py_is_py3k
... to only keep the Python 3 portions and drop the use of some
now-removed compatibility macros.
I did not update the configure script more than just removing the
explicit references to Python 2. We could maybe do more there, like
check the Python version and reject it if that version is not
supported. Otherwise (with this patch), things will only fail at
compile time, so it won't really be clear to the user that they are
trying to use an unsupported Python version. But I'm a bit lost in the
configure code that checks for Python, so I kept that for later.
Change-Id: I75b0f79c148afbe3c07ac664cfa9cade052c0c62
Diffstat (limited to 'gdb/python/py-membuf.c')
| -rw-r--r-- | gdb/python/py-membuf.c | 73 |
1 files changed, 1 insertions, 72 deletions
diff --git a/gdb/python/py-membuf.c b/gdb/python/py-membuf.c index 74cc0fb..348dfa3 100644 --- a/gdb/python/py-membuf.c +++ b/gdb/python/py-membuf.c @@ -54,15 +54,7 @@ gdbpy_buffer_to_membuf (gdb::unique_xmalloc_ptr<gdb_byte> buffer, membuf_obj->addr = address; membuf_obj->length = length; - PyObject *result; -#ifdef IS_PY3K - result = PyMemoryView_FromObject ((PyObject *) membuf_obj.get ()); -#else - result = PyBuffer_FromReadWriteObject ((PyObject *) membuf_obj.get (), 0, - Py_END_OF_BUFFER); -#endif - - return result; + return PyMemoryView_FromObject ((PyObject *) membuf_obj.get ()); } /* Destructor for gdb.Membuf objects. */ @@ -88,8 +80,6 @@ which is %s bytes long."), pulongest (membuf_obj->length)); } -#ifdef IS_PY3K - static int get_buffer (PyObject *self, Py_buffer *buf, int flags) { @@ -107,54 +97,6 @@ get_buffer (PyObject *self, Py_buffer *buf, int flags) return ret; } -#else - -static Py_ssize_t -get_read_buffer (PyObject *self, Py_ssize_t segment, void **ptrptr) -{ - membuf_object *membuf_obj = (membuf_object *) self; - - if (segment) - { - PyErr_SetString (PyExc_SystemError, - _("The memory buffer supports only one segment.")); - return -1; - } - - *ptrptr = membuf_obj->buffer; - - return membuf_obj->length; -} - -static Py_ssize_t -get_write_buffer (PyObject *self, Py_ssize_t segment, void **ptrptr) -{ - return get_read_buffer (self, segment, ptrptr); -} - -static Py_ssize_t -get_seg_count (PyObject *self, Py_ssize_t *lenp) -{ - if (lenp) - *lenp = ((membuf_object *) self)->length; - - return 1; -} - -static Py_ssize_t -get_char_buffer (PyObject *self, Py_ssize_t segment, char **ptrptr) -{ - void *ptr = nullptr; - Py_ssize_t ret; - - ret = get_read_buffer (self, segment, &ptr); - *ptrptr = (char *) ptr; - - return ret; -} - -#endif /* IS_PY3K */ - /* General Python initialization callback. */ int @@ -168,24 +110,11 @@ gdbpy_initialize_membuf (void) (PyObject *) &membuf_object_type); } -#ifdef IS_PY3K - static PyBufferProcs buffer_procs = { get_buffer }; -#else - -static PyBufferProcs buffer_procs = { - get_read_buffer, - get_write_buffer, - get_seg_count, - get_char_buffer -}; - -#endif /* IS_PY3K */ - PyTypeObject membuf_object_type = { PyVarObject_HEAD_INIT (nullptr, 0) "gdb.Membuf", /*tp_name*/ |