Eugene.Zelenko added a comment.
May be check belong to LLVM module?
================
Comment at: clang-tools-extra/clang-tidy/readability/QualifiedAutoCheck.cpp:69
+ if (auto Var = Result.Nodes.getNodeAs<VarDecl>("auto_ptr")) {
+
+ if (!Var->getType().getTypePtr()->getPointeeType().isConstQualified()) {
----------------
Unnecessary empty line.
================
Comment at:
clang-tools-extra/docs/clang-tidy/checks/readability-qualified-auto.rst:6
+
+`LLVM Coding Standards <https://llvm.org/docs/CodingStandards.html>`_ advises
to
+make it obvious if a auto typed variable is a pointer, constant pointer or
----------------
Please make first statement same as Release Notes.
================
Comment at:
clang-tools-extra/docs/clang-tidy/checks/readability-qualified-auto.rst:13
+.. code-block:: c++
+ for (auto &Data : MutatableContainer) {
+ change(Data);
----------------
Please separate with empty line.
================
Comment at:
clang-tools-extra/docs/clang-tidy/checks/readability-qualified-auto.rst:25
+ }
+
+
----------------
Unnecessary empty line.
================
Comment at:
clang-tools-extra/docs/clang-tidy/checks/readability-qualified-auto.rst:30
+.. code-block:: c++
+ for (auto &Data : MutatableContainer) {
+ change(Data);
----------------
Please separate with empty line.
================
Comment at:
clang-tools-extra/docs/clang-tidy/checks/readability-qualified-auto.rst:43
+
+
+This check helps to enforce this `LLVM Coding Standards recommendation
----------------
Unnecessary empty line.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72217/new/
https://reviews.llvm.org/D72217
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
