[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