lebedev.ri added inline comments.

================
Comment at: clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp:84
         "cppcoreguidelines-c-copy-assignment-signature");
+    CheckFactories.registerCheck<modernize::AvoidCArraysCheck>(
+        "cppcoreguidelines-avoid-c-arrays");
----------------
lebedev.ri wrote:
> JonasToth wrote:
> > lebedev.ri wrote:
> > > JonasToth wrote:
> > > > please conserve the alphabetical order here
> > > Sorted all the `CheckFactories.registerCheck<>();` lines.
> > the `avoid-*` checks seem to be displaced now.
> How it should be sorted? By the new check name? Or the internal class name?
> If it is not the latter, then sorting is really problematic as it can't be 
> automatized.
Maybe now is better?


https://reviews.llvm.org/D53771



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

Reply via email to