[all-commits] [llvm/llvm-project] d62a34: Add a test for adding and removing Listeners from ...
jimingham via All-commits
all-commits at lists.llvm.org
Fri Feb 7 17:54:33 PST 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: d62a343db23b7e8c3b5f2a0284bcacc10141a49e
https://github.com/llvm/llvm-project/commit/d62a343db23b7e8c3b5f2a0284bcacc10141a49e
Author: Jim Ingham <jingham at apple.com>
Date: 2020-02-07 (Fri, 07 Feb 2020)
Changed paths:
M lldb/packages/Python/lldbsuite/test/api/listeners/TestListener.py
Log Message:
-----------
Add a test for adding and removing Listeners from a BroadcasterManager.
Reid found a bug in removing Listeners from a BroadcasterManager:
https://reviews.llvm.org/D74010
The bug didn't affect the case where there was only one Listener
signed up for a BroadcasterManager, which was all the extant test
case tests. The driver also only uses one listener (the debugger)
for everything, so neither the test nor anything you do with lldb
command line would have triggered the bug.
This adds a couple more tests using more listeners, and adding and
removing them in a different way, which triggers a separate code path.
More information about the All-commits
mailing list