================ @@ -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