================ Comment at: clang-tidy/readability/CMakeLists.txt:10 @@ -9,2 +9,3 @@ ReadabilityTidyModule.cpp + RedundantInclude.cpp RedundantSmartptrGet.cpp ---------------- alexfh wrote: > I'm not a native speaker, but I have a feeling that "duplicate" is a more > suitable word here than "redundant". WDYT? Fixed.
================ Comment at: clang-tidy/readability/RedundantInclude.h:22 @@ +21,3 @@ +/// +/// Any change in the defined macros in the main file between \c #include +/// directives resets the list of includes used to check for duplicates. ---------------- alexfh wrote: > This description is somewhat confusing. Maybe something along the lines of > "Only consecutive #include directives without any other preprocessor > directives between them are analyzed"? Fixed. http://reviews.llvm.org/D7982 EMAIL PREFERENCES http://reviews.llvm.org/settings/panel/emailpreferences/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
