summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorINADA Naoki <songofacandy@gmail.com>2016-11-22 10:43:11 (GMT)
committerINADA Naoki <songofacandy@gmail.com>2016-11-22 10:43:11 (GMT)
commitff33c93f23646aacfac42d50d72961b3858f91e8 (patch)
treeb6f646d955e67d99d8dfe75d81ff4fd661bce601
parent1cd91a17a958a7c24cdc9976a950100fd7cd1117 (diff)
parentd7d2bc8798da3b083e383e949ba01d61b78e4e4d (diff)
downloadcpython-ff33c93f23646aacfac42d50d72961b3858f91e8.zip
cpython-ff33c93f23646aacfac42d50d72961b3858f91e8.tar.gz
cpython-ff33c93f23646aacfac42d50d72961b3858f91e8.tar.bz2
Issue #28023: Fix python-gdb.py didn't support new dict implementation
-rw-r--r--Lib/test/test_gdb.py9
-rw-r--r--Misc/NEWS2
-rwxr-xr-xTools/gdb/libpython.py28
3 files changed, 31 insertions, 8 deletions
diff --git a/Lib/test/test_gdb.py b/Lib/test/test_gdb.py
index 5fbf154..2bd4457 100644
--- a/Lib/test/test_gdb.py
+++ b/Lib/test/test_gdb.py
@@ -11,9 +11,6 @@ import sysconfig
import unittest
import locale
-# FIXME: issue #28023
-raise unittest.SkipTest("FIXME: issue #28023, compact dict (issue #27350) broke python-gdb.py")
-
# Is this Python configured to support threads?
try:
import _thread
@@ -296,9 +293,8 @@ class PrettyPrintTests(DebuggerTests):
'Verify the pretty-printing of dictionaries'
self.assertGdbRepr({})
self.assertGdbRepr({'foo': 'bar'}, "{'foo': 'bar'}")
- # PYTHONHASHSEED is need to get the exact item order
- if not sys.flags.ignore_environment:
- self.assertGdbRepr({'foo': 'bar', 'douglas': 42}, "{'douglas': 42, 'foo': 'bar'}")
+ # Python preserves insertion order since 3.6
+ self.assertGdbRepr({'foo': 'bar', 'douglas': 42}, "{'foo': 'bar', 'douglas': 42}")
def test_lists(self):
'Verify the pretty-printing of lists'
@@ -819,6 +815,7 @@ id(42)
)
self.assertIn('Garbage-collecting', gdb_output)
+ @unittest.skip("FIXME: builtin method is not shown in py-bt and py-bt-full")
@unittest.skipIf(python_is_optimized(),
"Python was compiled with optimizations")
# Some older versions of gdb will fail with
diff --git a/Misc/NEWS b/Misc/NEWS
index 72114e8..92fca57 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -493,6 +493,8 @@ Build
Tools/Demos
-----------
+- Issue #28023: Fix python-gdb.py didn't support new dict implementation.
+
- Issue #15369: The pybench and pystone microbenchmark have been removed from
Tools. Please use the new Python benchmark suite
https://github.com/python/performance which is more reliable and includes a
diff --git a/Tools/gdb/libpython.py b/Tools/gdb/libpython.py
index 3e95b44..964cc9f 100755
--- a/Tools/gdb/libpython.py
+++ b/Tools/gdb/libpython.py
@@ -666,8 +666,9 @@ class PyDictObjectPtr(PyObjectPtr):
'''
keys = self.field('ma_keys')
values = self.field('ma_values')
- for i in safe_range(keys['dk_size']):
- ep = keys['dk_entries'].address + i
+ entries, nentries = self._get_entries(keys)
+ for i in safe_range(nentries):
+ ep = entries[i]
if long(values):
pyop_value = PyObjectPtr.from_pyobject_ptr(values[i])
else:
@@ -707,6 +708,29 @@ class PyDictObjectPtr(PyObjectPtr):
pyop_value.write_repr(out, visited)
out.write('}')
+ def _get_entries(self, keys):
+ dk_size = int(keys['dk_size'])
+ try:
+ # <= Python 3.5
+ return keys['dk_entries'], dk_size
+ except gdb.error:
+ # >= Python 3.6
+ pass
+
+ if dk_size <= 0xFF:
+ offset = dk_size
+ elif dk_size <= 0xFFFF:
+ offset = 2 * dk_size
+ elif dk_size <= 0xFFFFFFFF:
+ offset = 4 * dk_size
+ else:
+ offset = 8 * dk_size
+
+ ent_ptr_t = gdb.lookup_type('PyDictKeyEntry').pointer()
+ ent_addr = int(keys['dk_indices']['as_1'].address) + offset
+ return gdb.Value(ent_addr).cast(ent_ptr_t), int(keys['dk_nentries'])
+
+
class PyListObjectPtr(PyObjectPtr):
_typename = 'PyListObject'