[PATCH] D85318: [clangd] Hide "swap if branch" tweak

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 5 08:24:53 PDT 2020


hokein created this revision.
hokein added a reviewer: kadircet.
Herald added subscribers: usaxena95, arphaman, jkorous.
Herald added a project: clang.
hokein requested review of this revision.
Herald added subscribers: MaskRay, ilya-biryukov.

This tweak is more like a demo, and doesn't provide much value in
practice.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85318

Files:
  clang-tools-extra/clangd/refactor/tweaks/SwapIfBranches.cpp


Index: clang-tools-extra/clangd/refactor/tweaks/SwapIfBranches.cpp
===================================================================
--- clang-tools-extra/clangd/refactor/tweaks/SwapIfBranches.cpp
+++ clang-tools-extra/clangd/refactor/tweaks/SwapIfBranches.cpp
@@ -40,6 +40,7 @@
   Expected<Effect> apply(const Selection &Inputs) override;
   std::string title() const override { return "Swap if branches"; }
   Intent intent() const override { return Refactor; }
+  bool hidden() const override { return true; }
 
 private:
   const IfStmt *If = nullptr;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85318.283251.patch
Type: text/x-patch
Size: 558 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200805/13f445ff/attachment.bin>


More information about the cfe-commits mailing list