[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