[PATCH] D122102: [clangd] Introduce "add subclass" tweak

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 21 12:27:16 PDT 2022


njames93 added a comment.

In its current incarnation, I struggle to see the use case for this. 
Most of the time Base classes live in header files are pretty much every time this tweak is applied you will then want to move the class.
The idea about just implementing the interface I had started in D94942 <https://reviews.llvm.org/D94942> but its been stale a while. However IMO that has more potential value.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122102



More information about the cfe-commits mailing list