[llvm-bugs] [Bug 32211] breaks clang-tidy annotation: // NOLINT(check)

via llvm-bugs llvm-bugs at lists.llvm.org
Fri Mar 10 03:22:01 PST 2017


Daniel Jasper <djasper at google.com> changed:

           What    |Removed                     |Added
             Status|NEW                         |RESOLVED
         Resolution|---                         |WORKSFORME

--- Comment #1 from Daniel Jasper <djasper at google.com> ---
This is not an easy decision.

In several projects, people think that you should add NOLINT-comments after
formatting. If formatting changes, you should also move the NOLINT afterwards.

However, whatever your preference, you can set this using clang-format's
CommentPragmas style option
(https://clang.llvm.org/docs/ClangFormatStyleOptions.html) to follow your own
preference. Thus I am going to mark this WORKSFORME.

You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20170310/7bac4d12/attachment.html>

More information about the llvm-bugs mailing list