diff options
| author | Etienne Bergeron <etienneb@google.com> | 2016-05-16 14:34:20 +0000 |
|---|---|---|
| committer | Etienne Bergeron <etienneb@google.com> | 2016-05-16 14:34:20 +0000 |
| commit | 8d73de9eac0e7bbfc913a0243b087925d6fe3d37 (patch) | |
| tree | cce218f404415058ab251486bef4ba40e53069be /clang-tools-extra/clang-tidy/utils/IncludeSorter.h | |
| parent | 4b9c0d4dcf4adcb869e41886df45e9e157566412 (diff) | |
| download | bcm5719-llvm-8d73de9eac0e7bbfc913a0243b087925d6fe3d37.tar.gz bcm5719-llvm-8d73de9eac0e7bbfc913a0243b087925d6fe3d37.zip | |
[clang-tidy] Cleanups utils files
Summary:
Cleanup some code by using appropriate APIs.
Some coding style cleanups.
There is no behavior changes.
- Function `IncludeSorter::CreateFixIt` can be replaced by `FixItHint::CreateReplacement`.
- Function `cleanPath` is a wrapper for `llvm::sys::path::remove_dots`.
Reviewers: alexfh
Subscribers: cfe-commits
Differential Revision: http://reviews.llvm.org/D20279
llvm-svn: 269656
Diffstat (limited to 'clang-tools-extra/clang-tidy/utils/IncludeSorter.h')
| -rw-r--r-- | clang-tools-extra/clang-tidy/utils/IncludeSorter.h | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/clang-tools-extra/clang-tidy/utils/IncludeSorter.h b/clang-tools-extra/clang-tidy/utils/IncludeSorter.h index 4939ffeec5c..7e941872a0c 100644 --- a/clang-tools-extra/clang-tidy/utils/IncludeSorter.h +++ b/clang-tools-extra/clang-tidy/utils/IncludeSorter.h @@ -15,7 +15,7 @@ namespace clang { namespace tidy { -namespace utils { +namespace utils { // Class used by IncludeSorterCallback and IncludeInserterCallback to record the // names of the inclusions in a given source file being processed and generate @@ -66,10 +66,6 @@ public: private: typedef SmallVector<SourceRange, 1> SourceRangeVector; - // Creates a fix-it for the given replacements. - // Takes the the source location that will be replaced, and the new text. - FixItHint CreateFixIt(SourceRange EditRange, const std::string &NewText); - const SourceManager *SourceMgr; const LangOptions *LangOpts; const IncludeStyle Style; |

