salman-javed-nz added inline comments.

================
Comment at: clang-tools-extra/docs/clang-tidy/index.rst:314
+are ignored here, as they would effectively re-activate the warning, causing
+confusion to the users.
 
----------------
There's something about this "causing confusion to the users" that my mind 
latches on to. I'm not sure what my problem with it is. Perhaps the sentence 
will read better if you just remove the 2nd half? 


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

https://reviews.llvm.org/D111208

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

Reply via email to