[all-commits] [llvm/llvm-project] 3e3e79: [lldb/test] Fix TestProgressReporting.py race issu...

Med Ismail Bennani via All-commits all-commits at lists.llvm.org
Wed Feb 23 16:49:03 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3e3e79a9e4c378b59f5f393f556e6a84edcd8898
      https://github.com/llvm/llvm-project/commit/3e3e79a9e4c378b59f5f393f556e6a84edcd8898
  Author: Med Ismail Bennani <medismail.bennani at gmail.com>
  Date:   2022-02-23 (Wed, 23 Feb 2022)

  Changed paths:
    M lldb/bindings/interface/SBDebugger.i
    M lldb/test/API/functionalities/progress_reporting/TestProgressReporting.py

  Log Message:
  -----------
  [lldb/test] Fix TestProgressReporting.py race issue with the event listener

This patch is a follow-up of D120100 to address some feedbacks from
@labath.

This should mainly fix the race issue with the even listener by moving
the listener setup to the main thread.

This also changes the SBDebugger::GetProgressFromEvent SWIG binding
arguments to be output only, so the user don't have to provide them.

Finally, this updates the test to check it the out arguments are returned
in a tuple and re-enables the test on all platforms.

Differential Revision: https://reviews.llvm.org/D120284

Signed-off-by: Med Ismail Bennani <medismail.bennani at gmail.com>




More information about the All-commits mailing list