0x8000-0000 added a comment.

In https://reviews.llvm.org/D49114#1159211, @Eugene.Zelenko wrote:

> C++ Core Guidelines contains ES.45: Avoid "magic constants"; use symbolic 
> constants 
> <https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#Res-magic>,
>  so I think check should be moved into cppcoreguidelines module.


The same guidelines existed in medical and aerospace environments for many 
years before C++ Core Guidelines. In fact the same checker should run just fine 
on C code (I'm not sure about Objective-C).

I am not opposed to moving it, if this is where the consensus leads - I know 
where to find it :)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D49114



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

Reply via email to