[Lldb-commits] [lldb] [lldb-dap][windows] fix lldb-dap's self.get_stdout() method (PR #181813)
Charles Zablit via lldb-commits
lldb-commits at lists.llvm.org
Wed Feb 18 04:07:01 PST 2026
================
@@ -215,10 +215,6 @@ Status ProcessWindows::DoLaunch(Module *exe_module,
if (error.Success())
SetID(launch_info.GetProcessID());
m_pty = launch_info.TakePTY();
- // At this point, Process owns the ConPTY. If ProcessLaunchInfo still has a
- // reference to it, it might get closed prematurely if another target is
- // created.
- assert(m_pty.use_count() == 1 && "More than one reference to the ConPTY");
----------------
charles-zablit wrote:
@Nerixyz I add to remove this because we do increment the reference count of the pty when we create a copy of the launch_info object. So `m_pty.use_count()` is 2 here.
https://github.com/llvm/llvm-project/pull/181813
More information about the lldb-commits
mailing list