JonasToth added a comment. Hi andobence,
thank you for the contribution. The check looks very good! Please add it to the release notes and synchronize the first line of the doc with the short sentence in the release notes describing what this check does. ================ Comment at: clang-tidy/modernize/DeprecatedIosBaseAliasesCheck.h:20 +/// This check warns the uses of the deprecated member types of ``std::ios_base`` +/// and replaces those that have a non - deprecated equivalent. +/// ---------------- I I think you can write `non-deprecated` without the whitespace. ================ Comment at: test/clang-tidy/modernize-deprecated-ios-base-aliases.cpp:14 + +template < + class CharT> ---------------- That format looks weird. Could you please run clang-format over the test as well? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51332 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
