[Python-checkins] gh-131763: Replace the redundant check with assert in remove_tools (#131765)

[gaogaotiantian]

https://github.com/python/cpython/commit/151d1bfd1bb7ca9a36bb0f2bd6df53d64a1ba2f2
commit: 151d1bfd1bb7ca9a36bb0f2bd6df53d64a1ba2f2
branch: main
author: Sergey Muraviov <smu...@mail.ru>
committer: gaogaotiantian <gaogaotiant...@hotmail.com>
date: 2025-03-26T18:36:04-04:00
summary:

gh-131763: Replace the redundant check with assert in remove_tools (#131765)

files:
M Python/instrumentation.c

diff --git a/Python/instrumentation.c b/Python/instrumentation.c
index bcf7267d23b2e3..979047cc6fe2cb 100644
--- a/Python/instrumentation.c
+++ b/Python/instrumentation.c
@@ -833,14 +833,15 @@ remove_tools(PyCodeObject * code, int offset, int event, 
int tools)
     assert(PY_MONITORING_IS_INSTRUMENTED_EVENT(event));
     assert(opcode_has_event(_Py_GetBaseCodeUnit(code, offset).op.code));
     _PyCoMonitoringData *monitoring = code->_co_monitoring;
+    assert(monitoring);
     bool should_de_instrument;
-    if (monitoring && monitoring->tools) {
+    if (monitoring->tools) {
         monitoring->tools[offset] &= ~tools;
         should_de_instrument = (monitoring->tools[offset] == 0);
     }
     else {
         /* Single tool */
-        uint8_t single_tool = 
code->_co_monitoring->active_monitors.tools[event];
+        uint8_t single_tool = monitoring->active_monitors.tools[event];
         assert(_Py_popcount32(single_tool) <= 1);
         should_de_instrument = ((single_tool & tools) == single_tool);
     }

_______________________________________________
Python-checkins mailing list -- python-checkins@python.org
To unsubscribe send an email to python-checkins-le...@python.org
https://mail.python.org/mailman3/lists/python-checkins.python.org/
Member address: arch...@mail-archive.com