================
@@ -31,8 +31,6 @@ class NoLintDirectiveHandler {
public:
NoLintDirectiveHandler();
~NoLintDirectiveHandler();
----------------
localspook wrote:
This check does backflips to keep the definition of `Impl` out of the header,
and this is one of those backflips. The destructor must be defined in the
`.cpp`, because the destructor of `Impl` (and by proxy, the destructor of
`std::unique_ptr<Impl>`) is only visible in the `.cpp`.
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
