Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Lib/bdb.py
Original file line number Diff line number Diff line change
Expand Up @@ -747,7 +747,7 @@ def clear_all_file_breaks(self, filename):
return 'There are no breakpoints in %s' % filename
for line in self.breaks[filename]:
blist = Breakpoint.bplist[filename, line]
for bp in blist:
for bp in blist[:]:
bp.deleteMe()
del self.breaks[filename]
return None
Expand Down
17 changes: 17 additions & 0 deletions Lib/test/test_bdb.py
Original file line number Diff line number Diff line change
Expand Up @@ -1232,6 +1232,23 @@ def test_next_to_botframe(self):


class TestRegressions(unittest.TestCase):
def test_clear_all_file_breaks_with_multiple_bps_same_line(self):
"""Regression test: gh-149015.
clear_all_file_breaks must remove all breakpoints even when
multiple breakpoints share the same (file, line)."""
dbg = Bdb()
src = canonic(__file__)
dbg.set_break(src, 10)
dbg.set_break(src, 10)
dbg.set_break(src, 10)
self.assertEqual(len(Breakpoint.bplist[(src, 10)]), 3)
dbg.clear_all_file_breaks(src)
self.assertNotIn((src, 10), Breakpoint.bplist)
for bp in list(Breakpoint.bpbynumber):
if bp is not None:
bp.deleteMe()


def test_format_stack_entry_no_lineno(self):
# See gh-101517
self.assertIn('Warning: lineno is None',
Expand Down
4 changes: 4 additions & 0 deletions Misc/NEWS.d/next/Library.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
bdb: Fix :meth:`bdb.Bdb.clear_all_file_breaks` to not skip breakpoints when multiple
breakpoints share the same file and line. Patch by Aman Sachan.

.. bpo-149015.
Loading