[PATCH] D55545: Allow IncludeSorter to use #import for Objective-C files

2018-12-13 Thread Ben Hamilton via Phabricator via cfe-commits
benhamilton added inline comments. Comment at: clang-tidy/utils/IncludeSorter.cpp:116 bool IsAngled) { + std::string IncludeDirective = LangOpts->ObjC ? "#import " : "#include "; std::string IncludeStmt =

[PATCH] D55545: Allow IncludeSorter to use #import for Objective-C files

2018-12-13 Thread Haojian Wu via Phabricator via cfe-commits
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 IncludeDirectiv

[PATCH] D55545: Allow IncludeSorter to use #import for Objective-C files

2018-12-10 Thread Joe Turner via Phabricator via cfe-commits
compositeprimes created this revision. compositeprimes added reviewers: klimek, alexfh. Herald added a subscriber: cfe-commits. The change infers whether a source file is Objective-C using LangOpts, and if it detects Objective-C, it uses "#import" instead of "#include" for all inserted imports.