massberg marked 4 inline comments as done. massberg added inline comments.
================ Comment at: clang-tidy/modernize/AvoidFunctionalCheck.h:19 + +/// Check for several deprecated types and classes from <functional> header +/// ---------------- massberg wrote: > aaron.ballman wrote: > > Missing full stop at the end of the sentence. > > > > Why should this modernize check be limited to `<functional>`? Just like we > > have a "deprecated headers" check, perhaps this should be a "deprecated > > APIs" check? > Added full stop. > > I'm not sure if this check should be limited to <functional> or be extended > to a full 'deprecated API' check. > This change is just a start, several more types and classes which are removed > from <functional> will follow, e.g: > > - std::ptr_fun, std::mem_fun, std::mem_fun_ref > - std::bind1st, std::bind2nd > - std::unary_function, std::binary_function > - std::pointer_to_unary_function, std::pointer_to_binary_function, > std::mem_fun_t, std::mem_fun1_t, std::const_mem_fun_t, > - std::const_mem_fun1_t, std::mem_fun_ref_t, std::mem_fun1_ref_t, > std::const_mem_fun_ref_t, std::const_mem_fun1_ref_t > - std::binder1st, std::binder2nd > > As these are a bunch of functions and types, in my eyes a check just for > <functional> is fine. But I'm also fine with a general 'deprecated API' check. > Alex, can you comment on this? There are already other checks for functions which are removed in C++17 like modernize-replace-random-shuffle. So I think having an separate check for functions and types removed from <functional> would be OK. https://reviews.llvm.org/D42730 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits