================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to