[all-commits] [llvm/llvm-project] 83c81c: [lldb] Make Fix-Its also apply to top-level expres...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Mon Mar 30 02:51:17 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 83c81c0a469482888482983c302c09c02680ae7c
      https://github.com/llvm/llvm-project/commit/83c81c0a469482888482983c302c09c02680ae7c
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2020-03-30 (Mon, 30 Mar 2020)

  Changed paths:
    M lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp
    M lldb/test/API/commands/expression/fixits/TestFixIts.py

  Log Message:
  -----------
  [lldb] Make Fix-Its also apply to top-level expressions

Summary:
Currently top-level expressions won't automatically get Fix-Its applied. The reason
for that is that we only set the `m_fixed_text` member if we have a wrapping
source code (I.e. `m_source_code` is not zero and is wrapping some expressions).

This patch just always sets `m_fixed_text` to get this working.

Reviewers: labath, jingham

Reviewed By: labath

Subscribers: JDevlieghere

Differential Revision: https://reviews.llvm.org/D77042




More information about the All-commits mailing list