hokein added a comment.
Please add a unittest for it, you can add it in
`unittests/clang-tidy/IncludeInserterTest.cpp`.
================
Comment at: clang-tidy/utils/IncludeSorter.cpp:116
bool IsAngled) {
+ std::string IncludeDirective = LangOpts->ObjC ? "#import " : "#include ";
std::string IncludeStmt =
----------------
What about the ObjC++? The current behavior is always using `#import`, google
objc style guide says `#import Objective-C and Objective-C++ headers, and
#include C/C++ headers.`, I don't think we have a way to do it smartly.
Maybe a conservative way is to use `#import ` for ObjC only?
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55545/new/
https://reviews.llvm.org/D55545
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
