summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexandr Mitin <64941904+Alexandr153@users.noreply.github.com>2024-08-28 06:15:34 (GMT)
committerGitHub <noreply@github.com>2024-08-28 06:15:34 (GMT)
commit6a7765b9fad8bf67e2c118b637a516c5e6c42349 (patch)
tree5b4b7037e600a93f346308aed4480bb696bc3c0d
parent2231286d78d328c2f575e0b05b16fe447d1656d6 (diff)
downloadcpython-6a7765b9fad8bf67e2c118b637a516c5e6c42349.zip
cpython-6a7765b9fad8bf67e2c118b637a516c5e6c42349.tar.gz
cpython-6a7765b9fad8bf67e2c118b637a516c5e6c42349.tar.bz2
gh-123363: Show string value of CONTAINS_OP oparg in dis (#123387)
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
-rw-r--r--Lib/dis.py3
-rw-r--r--Lib/test/test_dis.py9
-rw-r--r--Misc/NEWS.d/next/Library/2024-08-27-12-11-00.gh-issue-123363.gKuJp6.rst2
3 files changed, 14 insertions, 0 deletions
diff --git a/Lib/dis.py b/Lib/dis.py
index bdac296..f8832b3 100644
--- a/Lib/dis.py
+++ b/Lib/dis.py
@@ -52,6 +52,7 @@ LOAD_FAST_LOAD_FAST = opmap['LOAD_FAST_LOAD_FAST']
STORE_FAST_LOAD_FAST = opmap['STORE_FAST_LOAD_FAST']
STORE_FAST_STORE_FAST = opmap['STORE_FAST_STORE_FAST']
IS_OP = opmap['IS_OP']
+CONTAINS_OP = opmap['CONTAINS_OP']
CACHE = opmap["CACHE"]
@@ -632,6 +633,8 @@ class ArgResolver:
argrepr = _special_method_names[arg]
elif deop == IS_OP:
argrepr = 'is not' if argval else 'is'
+ elif deop == CONTAINS_OP:
+ argrepr = 'not in' if argval else 'in'
return argval, argrepr
def get_instructions(x, *, first_line=None, show_caches=None, adaptive=False):
diff --git a/Lib/test/test_dis.py b/Lib/test/test_dis.py
index ab0fcee..bccd218 100644
--- a/Lib/test/test_dis.py
+++ b/Lib/test/test_dis.py
@@ -2037,6 +2037,15 @@ class InstructionTests(InstructionTestCase):
dis.dis("a is not b", file=output, show_caches=True)
self.assertIn("IS_OP 1 (is not)", output.getvalue())
+ def test_contains_op_format(self):
+ output = io.StringIO()
+ dis.dis("a in b", file=output, show_caches=True)
+ self.assertIn("CONTAINS_OP 0 (in)", output.getvalue())
+
+ output = io.StringIO()
+ dis.dis("a not in b", file=output, show_caches=True)
+ self.assertIn("CONTAINS_OP 1 (not in)", output.getvalue())
+
def test_baseopname_and_baseopcode(self):
# Standard instructions
for name, code in dis.opmap.items():
diff --git a/Misc/NEWS.d/next/Library/2024-08-27-12-11-00.gh-issue-123363.gKuJp6.rst b/Misc/NEWS.d/next/Library/2024-08-27-12-11-00.gh-issue-123363.gKuJp6.rst
new file mode 100644
index 0000000..c1f92c4
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2024-08-27-12-11-00.gh-issue-123363.gKuJp6.rst
@@ -0,0 +1,2 @@
+Show string value of :opcode:`CONTAINS_OP` oparg in :mod:`dis` output.
+Patch by Alexandr153.