[cfe-dev] Why does clang-tidy recommend deleted member functions should be public?

[Vadim Peretokin via cfe-dev]

The modernize-use-equals-delete check despite not saying much about it in
the docs thinks that deleted member functions should be public (
https://github.com/llvm-mirror/clang-tools-extra/blob/master/test/clang-tidy/modernize-use-equals-delete.cpp#L145)
and it's thus going off on Qt's Q_DISABLE_COPY (
http://doc.qt.io/qt-5/qobject.html#Q_DISABLE_COPY) macro.

The guidelines from Qt are that this macro be private, not public, and it's
unclear to us why clang-tidy thinks so. Could anyone shed some light on the
reasoning?

Thanks,
Vadim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20171031/3cef2f94/attachment.html>