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