[Lldb-commits] [PATCH] D144688: [lldb] Fix watchpoint command function stopping behavior

Jim Ingham via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 24 11:27:01 PST 2023


jingham requested changes to this revision.
jingham added a comment.
This revision now requires changes to proceed.

IIUC, not removing elements from the internal dict will mean cross-contamination between the python states when you have multiple concurrent debuggers, which for instance Xcode does a lot.  New variables would appear in the other session, but not linked to this debugger.  That could cause subtle problems, which it's worth doing some extra work to avoid.

We should also make sure in the test that a watchpoint command that returns nothing still works.  That's suppose to be the equivalent of returning True.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144688/new/

https://reviews.llvm.org/D144688



More information about the lldb-commits mailing list