diff options
| author | Alexander Kornienko <alexfh@google.com> | 2015-01-20 09:48:51 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2015-01-20 09:48:51 +0000 |
| commit | 9115a3af0213100a907d8de39e906eb9959740d5 (patch) | |
| tree | 5651f45a703e09bc06f73b7afdc54bb80d3a8fc5 /clang-tools-extra/clang-tidy/ClangTidyOptions.cpp | |
| parent | bb917683b6406bf3bd2f5da38511695e69400755 (diff) | |
| download | bcm5719-llvm-9115a3af0213100a907d8de39e906eb9959740d5.tar.gz bcm5719-llvm-9115a3af0213100a907d8de39e906eb9959740d5.zip | |
[clang-tidy] Make ClangTidyOptionsProvider::getOptions return by value.
Returning by reference limits possible implementations and doesn't bring any
benefits as all callers make copies of the returned value anyway.
llvm-svn: 226553
Diffstat (limited to 'clang-tools-extra/clang-tidy/ClangTidyOptions.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/ClangTidyOptions.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/ClangTidyOptions.cpp b/clang-tools-extra/clang-tidy/ClangTidyOptions.cpp index fc0afb1d81d..997529392ff 100644 --- a/clang-tools-extra/clang-tidy/ClangTidyOptions.cpp +++ b/clang-tools-extra/clang-tidy/ClangTidyOptions.cpp @@ -159,7 +159,7 @@ FileOptionsProvider::FileOptionsProvider( // FIXME: This method has some common logic with clang::format::getStyle(). // Consider pulling out common bits to a findParentFileWithName function or // similar. -const ClangTidyOptions &FileOptionsProvider::getOptions(StringRef FileName) { +ClangTidyOptions FileOptionsProvider::getOptions(StringRef FileName) { DEBUG(llvm::dbgs() << "Getting options for file " << FileName << "...\n"); SmallString<256> FilePath(FileName); |
