0x8000-0000 marked 3 inline comments as done.
0x8000-0000 added inline comments.


================
Comment at: test/clang-tidy/readability-magic-numbers.cpp:16
+void BuggyFunction() {
+  int BadLocalInt = 6;
+  // CHECK-MESSAGES: :[[@LINE-1]]:21: warning: magic number integer literal 6 
[readability-magic-numbers]
----------------
aaron.ballman wrote:
> 0x8000-0000 wrote:
> > 0x8000-0000 wrote:
> > > Quuxplusone wrote:
> > > > Please add test cases immediately following this one, for
> > > > 
> > > >     const int BadLocalConstInt = 6;
> > > >     constexpr int BadLocalConstexprInt = 6;
> > > >     static const int BadLocalStaticConstInt = 6;
> > > >     static constexpr int BadLocalStaticConstexprInt = 6;
> > > > 
> > > > (except of course changing "Bad" to "Good" in any cases where 6 is 
> > > > acceptable as an initializer).
> > > > 
> > > > Also
> > > > 
> > > >     std::vector<int> BadLocalVector(6);
> > > >     const std::vector<int> BadLocalConstVector(6);
> > > > 
> > > > etc. etc.
> > > Again... all the "const .* (\d)+" patterns should be acceptable. We're 
> > > initializing a constant. Would you prefer an explicit option?
> > I have  template and constructor arguments already in the test. I have 
> > tried including <vector> but somehow it is not found and the std::vector is 
> > reported as an error in itself.
> Tests need to be hermetic and cannot rely on STL headers or other system 
> headers. Basically, you have to replicate the contents of <vector> (or 
> whatever) within the test file for whatever you're trying to test.
Ok - we have that already and there's nothing magic about std::vector or 
std::array.


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