xgsa marked 2 inline comments as done.
xgsa added inline comments.

================
Comment at: docs/clang-tidy/index.rst:254-255
+While :program:`clang-tidy` diagnostics are intended to call out code that does
+not adhere to a coding standard, or is otherwise problematic in some way, there
+are times when it is more appropriate to silence the diagnostic instead of 
+changing the semantics of the code. The preferable way of doing this is using
----------------
aaron.ballman wrote:
> I would reword this somewhat now. I would put a period before ", there are 
> times" and then move that whole "there are times" clause below.
I tried to move the "there are times"-clause below, but in this case "The 
preferable way of doing this is using..." becomes unclear, because it tries to 
explain the way of doing something without naming the purpose that is expected 
to achieve. So I suppose, it is necessary to place "However, there are times 
when it is more appropriate to silence the diagnostic instead of changing the 
semantics of the code" here. Am I missing something?


https://reviews.llvm.org/D40671



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to