Eugene.Zelenko added inline comments.

================
Comment at: clang-tidy/readability/ConstValueReturnCheck.cpp:54
+
+namespace {
+
----------------
lebedev.ri wrote:
> Extend the namespace /\ above, so that function is also covered?
Anonymous namespaces are only for class/struct declarations. See LLVM coding 
guidelines.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53025



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

Reply via email to