[clang-tidy] Make ClangTidyOptionsProvider::getOptions return by value.
authorAlexander Kornienko <alexfh@google.com>
Tue, 20 Jan 2015 09:48:51 +0000 (09:48 +0000)
committerAlexander Kornienko <alexfh@google.com>
Tue, 20 Jan 2015 09:48:51 +0000 (09:48 +0000)
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

clang-tools-extra/clang-tidy/ClangTidyOptions.cpp
clang-tools-extra/clang-tidy/ClangTidyOptions.h

index fc0afb1..9975293 100644 (file)
@@ -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);
 
index 52adcae..9f0521b 100644 (file)
@@ -95,7 +95,7 @@ public:
 
   /// \brief Returns options applying to a specific translation unit with the
   /// specified \p FileName.
-  virtual const ClangTidyOptions &getOptions(llvm::StringRef FileName) = 0;
+  virtual ClangTidyOptions getOptions(llvm::StringRef FileName) = 0;
 };
 
 /// \brief Implementation of the \c ClangTidyOptionsProvider interface, which
@@ -108,7 +108,7 @@ public:
   const ClangTidyGlobalOptions &getGlobalOptions() override {
     return GlobalOptions;
   }
-  const ClangTidyOptions &getOptions(llvm::StringRef /*FileName*/) override {
+  ClangTidyOptions getOptions(llvm::StringRef /*FileName*/) override {
     return DefaultOptions;
   }
 
@@ -187,7 +187,7 @@ public:
                       const ClangTidyOptions &OverrideOptions,
                       const ConfigFileHandlers &ConfigHandlers);
 
-  const ClangTidyOptions &getOptions(llvm::StringRef FileName) override;
+  ClangTidyOptions getOptions(llvm::StringRef FileName) override;
 
 private:
   /// \brief Try to read configuration files from \p Directory using registered