mib added inline comments.
================ Comment at: lldb/test/API/macosx/queues/TestQueues.py:133 + "The breakpoint for queue %s has not been hit" % (queue_breakpoint.GetQueueName())) + self.assertEqual(queue1_thread.GetStopReason(), 3, + "Queue %s is not stopped at breakpoint %d" % ---------------- Could you replace that with the enum variable ? ================ Comment at: lldb/test/API/macosx/queues/TestQueues.py:388 + # to be the name of the main thread + queue_breakpoint = lldbutil.run_to_name_breakpoint(self, "stopper", only_one_thread=False)[3] + queue_breakpoint.SetQueueName(main_thread.GetQueue().GetName()) ---------------- Do you really need a `main_thread` variable since `lldbutil.run_to_name_breakpoint` would return a tuple with `(target, process, thread, bkpt)` ? Is the thread returned here different from the `main_thread` you created above ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131605/new/ https://reviews.llvm.org/D131605 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
