[PATCH] D17990: [clang-tidy] minor improvements in modernise-deprecated-headers check
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 10 23:02:31 PST 2016
alexfh added a comment.
> + {"iso646.h", "ciso646"},
> Just delete #includes of this one. <ciso646> does nothing.
> + {"stdalign.h", "cstdalign"},
> + {"stdbool.h", "cstdbool"},
> We should just delete these two includes. These headers do nothing in C++.
These don't seem to be addressed. Can you at least leave a FIXME in the code and in the test?
http://reviews.llvm.org/D17990
More information about the cfe-commits
mailing list