sammccall marked 2 inline comments as done.
sammccall added inline comments.
================
Comment at: clangd/refactor/Tweak.h:104
+ /// Is this a 'hidden' tweak, which are off by default.
+ virtual bool hidden() const { return false; }
};
----------------
ilya-biryukov wrote:
> I wonder whether this should be a static method. WDYT?
>
> That would allow to even prevent calling `prepare()` on those tweaks.
> OTOH, we want `prepare()` should be fast and it shouldn't matter if that's
> the case.
We'd need to add criteria to prepareTweaks() as hidden() couldn't be checked
polymorphically on the results.
I think this is a good idea but should probably happen if/when we have a more
important reason to touch that API.
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62538/new/
https://reviews.llvm.org/D62538
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
