alexfh added inline comments. ================ Comment at: docs/clang-tidy/checks/modernize-deprecated-headers.rst:39 @@ +38,3 @@ +* `<string.h>` -> `<cstring>` +* `<tgmath.h>` -> `<ctgmath>` // deprecated since C++11 +* `<time.h>` -> `<ctime>` ---------------- alexfh wrote: > Not sure if "deprecated" is the right word here. The wording seems to be > slightly different: "This header simply includes <cmath> and <ccomplex>." What about this comment? (was on the' <tgmath.h> -> <ctgmath>` line)
================ Comment at: docs/clang-tidy/checks/modernize-deprecated-headers.rst:41 @@ +40,3 @@ +* `<time.h>` -> `<ctime>` +* `<uchar.h>` -> `<cuchar>` // deprecated since C++11 +* `<wchar.h>` -> `<cwchar>` ---------------- alexfh wrote: > It's not exactly "deprecated". It has appeared in C++11 and is not > particularly useful (just contains a couple of feature macros). Maybe we > should just remove #includes of this header as well. What about this comment? (was on the' <uchar.h> -> <cuchar>` line) http://reviews.llvm.org/D17990 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
