[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