Eugene.Zelenko added inline comments.
================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:260 +- Fixed an issue in :doc:`performance-noexcept-move-constructor + <clang-tidy/checks/performance/noexcept-move-constructor>` which resulted in false ---------------- AMS21 wrote: > Eugene.Zelenko wrote: > > Please keep alphabetical order (by check name) in this section. > Sure but I'm a bit confused. I assuming the check name is without the prefix > so in this case `noexcept-move-constructor`. So it should be after > `readability-avoid-underscore-in-googletest-name`. But why then is > `use-after-move` sorted before `identifier-naming`. > I'm probably missing something very obvious. Full check name (i.e. `performance-noexcept-move-constructor`) should be accounted. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146922/new/ https://reviews.llvm.org/D146922 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits