[Lldb-commits] [PATCH] D116788: [lldb] Set result error state in 'frame variable'

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jan 11 14:02:48 PST 2022


kastiglione added a comment.

> if I run “frame var” from HandleCommand in a script, with this change I can’t rely on the command status to tell whether the command output had anything interesting in it. That seems awkward.

This could be resolved by introducing a new enum value to indicate partial success, partial failure. I think that would be reasonable. I don't think that should block this change though, do you?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116788



More information about the lldb-commits mailing list