[PATCH] D54945: This commit adds a chapter about clang-tidy integrations

Thu Dec 6 06:08:20 PST 2018

alexfh added a comment.

In D54945#1318343 <https://reviews.llvm.org/D54945#1318343>, @MarinaKalashina wrote:

> @alexfh Thanks a lot for your patience and help. I've made another revision, now with the diff made by 'git show HEAD -U999999' to have the full context availlable.

I don't know how this happened, but the diff is still almost empty. This is what I get when I click on the "Download raw diff":

  Index: docs/clang-tidy/index.rst
  ===================================================================
  --- docs/clang-tidy/index.rst
  +++ docs/clang-tidy/index.rst
  @@ -403,7 +403,7 @@

       // Silent only the specified diagnostics for the next line
       // NOLINTNEXTLINE(google-explicit-constructor, google-runtime-int)
  -    Foo(bool param);
  +    Foo(bool param); 
     };

   The formal syntax of ``NOLINT``/``NOLINTNEXTLINE`` is the following:

It's definitely not a common problem, since there are other patches in review, which are displayed just fine.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54945/new/

https://reviews.llvm.org/D54945