This revision was automatically updated to reflect the committed changes. Closed by commit rL319785: [clang-tidy] adjust cppcoreguidelines-owning-memory documentation (authored by JonasToth).
Repository: rL LLVM https://reviews.llvm.org/D40829 Files: clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst Index: clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst =================================================================== --- clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst @@ -111,7 +111,11 @@ In template functions and classes, like in the example below, the information of the type aliases gets lost. Therefore using ``gsl::owner<T*>`` in a heavy templated code base might lead to false positives. -This limitation results in ``std::vector<gsl::owner<T*>>`` not being diagnosed correctly. + +Known code constructs that do not get diagnosed correctly are: + +- ``std::exchange`` +- ``std::vector<gsl::owner<T*>>`` .. code-block:: c++
Index: clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst =================================================================== --- clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst +++ clang-tools-extra/trunk/docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst @@ -111,7 +111,11 @@ In template functions and classes, like in the example below, the information of the type aliases gets lost. Therefore using ``gsl::owner<T*>`` in a heavy templated code base might lead to false positives. -This limitation results in ``std::vector<gsl::owner<T*>>`` not being diagnosed correctly. + +Known code constructs that do not get diagnosed correctly are: + +- ``std::exchange`` +- ``std::vector<gsl::owner<T*>>`` .. code-block:: c++
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
