[all-commits] [llvm/llvm-project] 696f97: [lldb] Set result error state in 'frame variable'
Dave Lee via All-commits
all-commits at lists.llvm.org
Sat Jan 15 14:20:39 PST 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 696f9706f34d76bca8f8d496eabe1a67b60e8a54
https://github.com/llvm/llvm-project/commit/696f9706f34d76bca8f8d496eabe1a67b60e8a54
Author: Dave Lee <davelee.com at gmail.com>
Date: 2022-01-15 (Sat, 15 Jan 2022)
Changed paths:
M lldb/source/Commands/CommandObjectFrame.cpp
Log Message:
-----------
[lldb] Set result error state in 'frame variable'
Ensure that errors in `frame variable` are reflected in result object.
The statistics for `frame variable` show invocations as being successful, even
when executing one of the error paths.
This change replaces `result.GetErrorStream()` with `result.AppendError()`,
which also sets the status to `eReturnStatusFailed`.
Differential Revision: https://reviews.llvm.org/D116788
Recommitting after D116901 and D116863.
(cherry picked from commit 2c7d10c41278181e3e45c68f28b501cd95193a8a)
More information about the All-commits
mailing list