Index: docs/clang-tidy/checks/llvm-namespace-comment.rst =================================================================== --- docs/clang-tidy/checks/llvm-namespace-comment.rst +++ docs/clang-tidy/checks/llvm-namespace-comment.rst @@ -12,6 +12,19 @@ https://google.github.io/styleguide/cppguide.html#Namespaces +.. code-block:: c++ + + namespace n1 { + void f(); + } + + // becomes + + namespace n1 { + void f(); + } // namespace n1 + + Options ------- Index: docs/clang-tidy/checks/llvm-twine-local.rst =================================================================== --- docs/clang-tidy/checks/llvm-twine-local.rst +++ docs/clang-tidy/checks/llvm-twine-local.rst @@ -6,3 +6,11 @@ Looks for local ``Twine`` variables which are prone to use after frees and should be generally avoided. + +.. code-block:: c++ + + static Twine Moo = Twine("bark") + "bah"; + + // becomes + + static std::string Moo = (Twine("bark") + "bah").str(); Index: docs/clang-tidy/checks/misc-inefficient-algorithm.rst =================================================================== --- docs/clang-tidy/checks/misc-inefficient-algorithm.rst +++ docs/clang-tidy/checks/misc-inefficient-algorithm.rst @@ -9,3 +9,21 @@ Associative containers implements some of the algorithms as methods which should be preferred to the algorithms in the algorithm header. The methods can take advanatage of the order of the elements. + +.. code-block:: c++ + + std::set s; + auto it = std::find(s.begin(), s.end(), 43); + + // becomes + + auto it = s.find(43); + +.. code-block:: c++ + + std::set s; + auto c = count(s.begin(), s.end(), 43); + + // becomes + + auto c = s.count(43); Index: docs/clang-tidy/checks/misc-unused-parameters.rst =================================================================== --- docs/clang-tidy/checks/misc-unused-parameters.rst +++ docs/clang-tidy/checks/misc-unused-parameters.rst @@ -5,3 +5,21 @@ Finds unused parameters and fixes them, so that `-Wunused-parameter` can be turned on. + +.. code-block:: c++ + + void a(int i) {} + + // becomes + + void a(int /*i*/) {} + + +.. code-block:: c++ + + static void staticFunctionA(int i); + static void staticFunctionA(int i) {} + + // becomes + + static void staticFunctionA() Index: docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst =================================================================== --- docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst +++ docs/clang-tidy/checks/performance-type-promotion-in-math-fn.rst @@ -9,3 +9,13 @@ For example, warns on ``::sin(0.f)``, because this funciton's parameter is a double. You probably meant to call ``std::sin(0.f)`` (in C++), or ``sinf(0.f)`` (in C). + +.. code-block:: c++ + + float a; + asin(a); + + // becomes + + float a; + std::asin(a); Index: docs/clang-tidy/checks/readability-redundant-string-init.rst =================================================================== --- docs/clang-tidy/checks/readability-redundant-string-init.rst +++ docs/clang-tidy/checks/readability-redundant-string-init.rst @@ -12,3 +12,8 @@ // Initializing string with empty string literal is unnecessary. std::string a = ""; std::string b(""); + + // becomes + + std::string a; + std::string b; Index: docs/clang-tidy/checks/readability-uniqueptr-delete-release.rst =================================================================== --- docs/clang-tidy/checks/readability-uniqueptr-delete-release.rst +++ docs/clang-tidy/checks/readability-uniqueptr-delete-release.rst @@ -5,3 +5,11 @@ Replace ``delete .release()`` with `` = nullptr``. The latter is shorter, simpler and does not require use of raw pointer APIs. + +.. code-block:: c++ + + delete P.release(); + + // becomes + + P = nullptr;