lebedev.ri added inline comments.
================ Comment at: clang-tidy/modernize/UseAutoCheck.cpp:290 + : ClangTidyCheck(Name, Context), RemoveStars(Options.get("RemoveStars", 0)), + MinTypeNameLength(Options.get("MinTypeNameLength", 0)) {} ---------------- alexfh wrote: > Maybe make the default 5? Or does anyone really want to replace > `int/long/char/bool/...` with `auto`? That might be a bit surprising behavioral change.. At least it should be spelled out in the release notes. (my 5 cent: defaulting to 0 would be best) https://reviews.llvm.org/D45405 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits