diff options
author | Alexandr Mitin <64941904+Alexandr153@users.noreply.github.com> | 2024-08-28 06:15:34 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-08-28 06:15:34 (GMT) |
commit | 6a7765b9fad8bf67e2c118b637a516c5e6c42349 (patch) | |
tree | 5b4b7037e600a93f346308aed4480bb696bc3c0d | |
parent | 2231286d78d328c2f575e0b05b16fe447d1656d6 (diff) | |
download | cpython-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.py | 3 | ||||
-rw-r--r-- | Lib/test/test_dis.py | 9 | ||||
-rw-r--r-- | Misc/NEWS.d/next/Library/2024-08-27-12-11-00.gh-issue-123363.gKuJp6.rst | 2 |
3 files changed, 14 insertions, 0 deletions
@@ -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. |