[all-commits] [llvm/llvm-project] 2eb9cc: [clangd] Handle destructors in DefineOutline tweak

Nathan James via All-commits all-commits at lists.llvm.org
Fri Apr 14 09:52:03 PDT 2023

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2eb9cc76a6c0b61fcf75b747258e27b7b7683ca1
  Author: Nathan James <n.james93 at hotmail.co.uk>
  Date:   2023-04-14 (Fri, 14 Apr 2023)

  Changed paths:
    M clang-tools-extra/clangd/refactor/tweaks/DefineOutline.cpp
    M clang-tools-extra/clangd/unittests/tweaks/DefineOutlineTests.cpp

  Log Message:
  [clangd] Handle destructors in DefineOutline tweak

Fix destructors being incorrectly defined in the DefineOutline tweak
Currently it doesn't prepend the class name to the destructor
class A { ~A() {} };
// Destructor definition after outline
~A() {}
// After this fix
A::~A() {}

Reviewed By: kadircet

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

More information about the All-commits mailing list