[clang-tools-extra] [clang-tidy][NFC] Do less unnecessary work in `NoLintDirectiveHandler` (PR #147553)
Nicolas van Kempen via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 9 19:21:25 PDT 2025
================
@@ -31,8 +31,6 @@ class NoLintDirectiveHandler {
public:
NoLintDirectiveHandler();
~NoLintDirectiveHandler();
----------------
nicovank wrote:
This destructor being explicit also makes the class non-movable. It's `= default;` anyway. (Also needs a change in `.cpp`.)
```suggestion
```
Or if the class should be non-movable then it should be explicit (I haven't read the PR more in-depth).
https://github.com/llvm/llvm-project/pull/147553
More information about the cfe-commits
mailing list