Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 1 | //===--- ClangTidyOptions.cpp - clang-tidy ----------------------*- C++ -*-===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | |
| 10 | #include "ClangTidyOptions.h" |
Alexander Kornienko | 1efc425 | 2014-10-16 11:27:57 +0000 | [diff] [blame] | 11 | #include "ClangTidyModuleRegistry.h" |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 12 | #include "clang/Basic/LLVM.h" |
| 13 | #include "llvm/ADT/SmallString.h" |
Hans Wennborg | fece4c6 | 2014-09-04 22:41:03 +0000 | [diff] [blame] | 14 | #include "llvm/Support/Errc.h" |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 15 | #include "llvm/Support/Debug.h" |
| 16 | #include "llvm/Support/FileSystem.h" |
| 17 | #include "llvm/Support/Path.h" |
| 18 | #include "llvm/Support/raw_ostream.h" |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 19 | #include "llvm/Support/YAMLTraits.h" |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 20 | #include <utility> |
| 21 | |
| 22 | #define DEBUG_TYPE "clang-tidy-options" |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 23 | |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 24 | using clang::tidy::ClangTidyOptions; |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 25 | using clang::tidy::FileFilter; |
| 26 | |
| 27 | LLVM_YAML_IS_FLOW_SEQUENCE_VECTOR(FileFilter) |
| 28 | LLVM_YAML_IS_FLOW_SEQUENCE_VECTOR(FileFilter::LineRange) |
Benjamin Kramer | 39e40cc | 2014-09-23 14:46:55 +0000 | [diff] [blame] | 29 | LLVM_YAML_IS_SEQUENCE_VECTOR(ClangTidyOptions::StringPair) |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 30 | |
| 31 | namespace llvm { |
| 32 | namespace yaml { |
| 33 | |
| 34 | // Map std::pair<int, int> to a JSON array of size 2. |
| 35 | template <> struct SequenceTraits<FileFilter::LineRange> { |
| 36 | static size_t size(IO &IO, FileFilter::LineRange &Range) { |
| 37 | return Range.first == 0 ? 0 : Range.second == 0 ? 1 : 2; |
| 38 | } |
| 39 | static unsigned &element(IO &IO, FileFilter::LineRange &Range, size_t Index) { |
| 40 | if (Index > 1) |
| 41 | IO.setError("Too many elements in line range."); |
| 42 | return Index == 0 ? Range.first : Range.second; |
| 43 | } |
| 44 | }; |
| 45 | |
| 46 | template <> struct MappingTraits<FileFilter> { |
| 47 | static void mapping(IO &IO, FileFilter &File) { |
| 48 | IO.mapRequired("name", File.Name); |
| 49 | IO.mapOptional("lines", File.LineRanges); |
| 50 | } |
| 51 | static StringRef validate(IO &io, FileFilter &File) { |
| 52 | if (File.Name.empty()) |
| 53 | return "No file name specified"; |
| 54 | for (const FileFilter::LineRange &Range : File.LineRanges) { |
| 55 | if (Range.first <= 0 || Range.second <= 0) |
| 56 | return "Invalid line range"; |
| 57 | } |
| 58 | return StringRef(); |
| 59 | } |
| 60 | }; |
| 61 | |
Alexander Kornienko | 6e0cbc8 | 2014-09-12 08:53:36 +0000 | [diff] [blame] | 62 | template <> struct MappingTraits<ClangTidyOptions::StringPair> { |
| 63 | static void mapping(IO &IO, ClangTidyOptions::StringPair &KeyValue) { |
| 64 | IO.mapRequired("key", KeyValue.first); |
| 65 | IO.mapRequired("value", KeyValue.second); |
| 66 | } |
| 67 | }; |
| 68 | |
| 69 | struct NOptionMap { |
| 70 | NOptionMap(IO &) {} |
| 71 | NOptionMap(IO &, const ClangTidyOptions::OptionMap &OptionMap) |
| 72 | : Options(OptionMap.begin(), OptionMap.end()) {} |
| 73 | ClangTidyOptions::OptionMap denormalize(IO &) { |
| 74 | ClangTidyOptions::OptionMap Map; |
| 75 | for (const auto &KeyValue : Options) |
| 76 | Map[KeyValue.first] = KeyValue.second; |
| 77 | return Map; |
| 78 | } |
| 79 | std::vector<ClangTidyOptions::StringPair> Options; |
| 80 | }; |
| 81 | |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 82 | template <> struct MappingTraits<ClangTidyOptions> { |
| 83 | static void mapping(IO &IO, ClangTidyOptions &Options) { |
Alexander Kornienko | 6e0cbc8 | 2014-09-12 08:53:36 +0000 | [diff] [blame] | 84 | MappingNormalization<NOptionMap, ClangTidyOptions::OptionMap> NOpts( |
| 85 | IO, Options.CheckOptions); |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 86 | IO.mapOptional("Checks", Options.Checks); |
| 87 | IO.mapOptional("HeaderFilterRegex", Options.HeaderFilterRegex); |
| 88 | IO.mapOptional("AnalyzeTemporaryDtors", Options.AnalyzeTemporaryDtors); |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 89 | IO.mapOptional("User", Options.User); |
Alexander Kornienko | 6e0cbc8 | 2014-09-12 08:53:36 +0000 | [diff] [blame] | 90 | IO.mapOptional("CheckOptions", NOpts->Options); |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 91 | } |
| 92 | }; |
| 93 | |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 94 | } // namespace yaml |
| 95 | } // namespace llvm |
| 96 | |
| 97 | namespace clang { |
| 98 | namespace tidy { |
| 99 | |
Alexander Kornienko | 1efc425 | 2014-10-16 11:27:57 +0000 | [diff] [blame] | 100 | ClangTidyOptions ClangTidyOptions::getDefaults() { |
| 101 | ClangTidyOptions Options; |
| 102 | Options.Checks = ""; |
| 103 | Options.HeaderFilterRegex = ""; |
| 104 | Options.AnalyzeTemporaryDtors = false; |
| 105 | Options.User = llvm::None; |
| 106 | for (ClangTidyModuleRegistry::iterator I = ClangTidyModuleRegistry::begin(), |
| 107 | E = ClangTidyModuleRegistry::end(); |
| 108 | I != E; ++I) |
| 109 | Options = Options.mergeWith(I->instantiate()->getModuleOptions()); |
| 110 | return Options; |
| 111 | } |
| 112 | |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 113 | ClangTidyOptions |
| 114 | ClangTidyOptions::mergeWith(const ClangTidyOptions &Other) const { |
| 115 | ClangTidyOptions Result = *this; |
| 116 | |
| 117 | // Merge comma-separated glob lists by appending the new value after a comma. |
| 118 | if (Other.Checks) |
| 119 | Result.Checks = |
| 120 | (Result.Checks && !Result.Checks->empty() ? *Result.Checks + "," : "") + |
| 121 | *Other.Checks; |
| 122 | |
| 123 | if (Other.HeaderFilterRegex) |
| 124 | Result.HeaderFilterRegex = Other.HeaderFilterRegex; |
| 125 | if (Other.AnalyzeTemporaryDtors) |
| 126 | Result.AnalyzeTemporaryDtors = Other.AnalyzeTemporaryDtors; |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 127 | if (Other.User) |
| 128 | Result.User = Other.User; |
Alexander Kornienko | 6e0cbc8 | 2014-09-12 08:53:36 +0000 | [diff] [blame] | 129 | |
| 130 | for (const auto &KeyValue : Other.CheckOptions) |
| 131 | Result.CheckOptions[KeyValue.first] = KeyValue.second; |
| 132 | |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 133 | return Result; |
| 134 | } |
| 135 | |
| 136 | FileOptionsProvider::FileOptionsProvider( |
| 137 | const ClangTidyGlobalOptions &GlobalOptions, |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 138 | const ClangTidyOptions &DefaultOptions, |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 139 | const ClangTidyOptions &OverrideOptions) |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 140 | : DefaultOptionsProvider(GlobalOptions, DefaultOptions), |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 141 | OverrideOptions(OverrideOptions) { |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 142 | CachedOptions[""] = DefaultOptions.mergeWith(OverrideOptions); |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 143 | } |
| 144 | |
| 145 | static const char ConfigFileName[] = ".clang-tidy"; |
| 146 | |
| 147 | // FIXME: This method has some common logic with clang::format::getStyle(). |
| 148 | // Consider pulling out common bits to a findParentFileWithName function or |
| 149 | // similar. |
| 150 | const ClangTidyOptions &FileOptionsProvider::getOptions(StringRef FileName) { |
| 151 | DEBUG(llvm::dbgs() << "Getting options for file " << FileName << "...\n"); |
| 152 | SmallString<256> FilePath(FileName); |
| 153 | |
| 154 | if (std::error_code EC = llvm::sys::fs::make_absolute(FilePath)) { |
| 155 | llvm::errs() << "Can't make absolute path from " << FileName << ": " |
| 156 | << EC.message() << "\n"; |
| 157 | // FIXME: Figure out what to do. |
| 158 | } else { |
| 159 | FileName = FilePath; |
| 160 | } |
| 161 | |
| 162 | // Look for a suitable configuration file in all parent directories of the |
| 163 | // file. Start with the immediate parent directory and move up. |
| 164 | StringRef Path = llvm::sys::path::parent_path(FileName); |
| 165 | for (StringRef CurrentPath = Path;; |
| 166 | CurrentPath = llvm::sys::path::parent_path(CurrentPath)) { |
| 167 | llvm::ErrorOr<ClangTidyOptions> Result = std::error_code(); |
| 168 | |
| 169 | auto Iter = CachedOptions.find(CurrentPath); |
| 170 | if (Iter != CachedOptions.end()) |
| 171 | Result = Iter->second; |
| 172 | |
| 173 | if (!Result) |
| 174 | Result = TryReadConfigFile(CurrentPath); |
| 175 | |
| 176 | if (Result) { |
| 177 | // Store cached value for all intermediate directories. |
| 178 | while (Path != CurrentPath) { |
| 179 | DEBUG(llvm::dbgs() << "Caching configuration for path " << Path |
| 180 | << ".\n"); |
| 181 | CachedOptions.GetOrCreateValue(Path, *Result); |
| 182 | Path = llvm::sys::path::parent_path(Path); |
| 183 | } |
| 184 | return CachedOptions.GetOrCreateValue(Path, *Result).getValue(); |
| 185 | } |
Hans Wennborg | fece4c6 | 2014-09-04 22:41:03 +0000 | [diff] [blame] | 186 | if (Result.getError() != llvm::errc::no_such_file_or_directory) { |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 187 | llvm::errs() << "Error reading " << ConfigFileName << " from " << Path |
| 188 | << ": " << Result.getError().message() << "\n"; |
| 189 | } |
| 190 | } |
| 191 | } |
| 192 | |
| 193 | llvm::ErrorOr<ClangTidyOptions> |
| 194 | FileOptionsProvider::TryReadConfigFile(StringRef Directory) { |
| 195 | assert(!Directory.empty()); |
| 196 | |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 197 | if (!llvm::sys::fs::is_directory(Directory)) |
Hans Wennborg | fece4c6 | 2014-09-04 22:41:03 +0000 | [diff] [blame] | 198 | return make_error_code(llvm::errc::not_a_directory); |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 199 | |
| 200 | SmallString<128> ConfigFile(Directory); |
| 201 | llvm::sys::path::append(ConfigFile, ".clang-tidy"); |
| 202 | DEBUG(llvm::dbgs() << "Trying " << ConfigFile << "...\n"); |
| 203 | |
| 204 | bool IsFile = false; |
| 205 | // Ignore errors from is_regular_file: we only need to know if we can read |
| 206 | // the file or not. |
| 207 | llvm::sys::fs::is_regular_file(Twine(ConfigFile), IsFile); |
| 208 | |
| 209 | if (!IsFile) |
Hans Wennborg | fece4c6 | 2014-09-04 22:41:03 +0000 | [diff] [blame] | 210 | return make_error_code(llvm::errc::no_such_file_or_directory); |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 211 | |
| 212 | llvm::ErrorOr<std::unique_ptr<llvm::MemoryBuffer>> Text = |
| 213 | llvm::MemoryBuffer::getFile(ConfigFile.c_str()); |
| 214 | if (std::error_code EC = Text.getError()) |
| 215 | return EC; |
Alexander Kornienko | 6e0cbc8 | 2014-09-12 08:53:36 +0000 | [diff] [blame] | 216 | // Skip empty files, e.g. files opened for writing via shell output |
| 217 | // redirection. |
| 218 | if ((*Text)->getBuffer().empty()) |
| 219 | return make_error_code(llvm::errc::no_such_file_or_directory); |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 220 | llvm::ErrorOr<ClangTidyOptions> ParsedOptions = |
| 221 | parseConfiguration((*Text)->getBuffer()); |
| 222 | if (ParsedOptions) { |
| 223 | ClangTidyOptions Defaults = DefaultOptionsProvider::getOptions(Directory); |
| 224 | // Only use checks from the config file. |
| 225 | Defaults.Checks = None; |
| 226 | return Defaults.mergeWith(*ParsedOptions).mergeWith(OverrideOptions); |
| 227 | } |
| 228 | return ParsedOptions.getError(); |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 229 | } |
| 230 | |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 231 | /// \brief Parses -line-filter option and stores it to the \c Options. |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 232 | std::error_code parseLineFilter(StringRef LineFilter, |
Rafael Espindola | 15c5784 | 2014-06-12 13:32:11 +0000 | [diff] [blame] | 233 | clang::tidy::ClangTidyGlobalOptions &Options) { |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 234 | llvm::yaml::Input Input(LineFilter); |
| 235 | Input >> Options.LineFilter; |
| 236 | return Input.error(); |
| 237 | } |
| 238 | |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 239 | llvm::ErrorOr<ClangTidyOptions> parseConfiguration(StringRef Config) { |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 240 | llvm::yaml::Input Input(Config); |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 241 | ClangTidyOptions Options; |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 242 | Input >> Options; |
Alexander Kornienko | e995154 | 2014-09-24 18:36:03 +0000 | [diff] [blame] | 243 | if (Input.error()) |
| 244 | return Input.error(); |
| 245 | return Options; |
Alexander Kornienko | a469522 | 2014-06-05 13:31:45 +0000 | [diff] [blame] | 246 | } |
| 247 | |
Alexander Kornienko | d53d268 | 2014-09-04 14:23:36 +0000 | [diff] [blame] | 248 | std::string configurationAsText(const ClangTidyOptions &Options) { |
| 249 | std::string Text; |
| 250 | llvm::raw_string_ostream Stream(Text); |
| 251 | llvm::yaml::Output Output(Stream); |
| 252 | // We use the same mapping method for input and output, so we need a non-const |
| 253 | // reference here. |
| 254 | ClangTidyOptions NonConstValue = Options; |
| 255 | Output << NonConstValue; |
| 256 | return Stream.str(); |
| 257 | } |
| 258 | |
Alexander Kornienko | dad4acb | 2014-05-22 16:07:11 +0000 | [diff] [blame] | 259 | } // namespace tidy |
| 260 | } // namespace clang |