[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