Author: alexfh Date: Wed Sep 24 13:36:03 2014 New Revision: 218402 URL: http://llvm.org/viewvc/llvm-project?rev=218402&view=rev Log: Provide user name in ClangTidyOptions.
Summary: This adds the ClangTidyOptions::User field and fills it from the USER or the USERNAME environment variable, if possible. The FileOptionsProvider now takes "default" options instead of "fallback" options, as it now uses these when an option is not set in the configuration file (one exception is the checks list). Reviewers: bkramer, klimek Reviewed By: klimek Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D5440 Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.h clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.cpp clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.h clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp?rev=218402&r1=218401&r2=218402&view=diff ============================================================================== --- clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp (original) +++ clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.cpp Wed Sep 24 13:36:03 2014 @@ -85,6 +85,7 @@ template <> struct MappingTraits<ClangTi IO.mapOptional("Checks", Options.Checks); IO.mapOptional("HeaderFilterRegex", Options.HeaderFilterRegex); IO.mapOptional("AnalyzeTemporaryDtors", Options.AnalyzeTemporaryDtors); + IO.mapOptional("User", Options.User); IO.mapOptional("CheckOptions", NOpts->Options); } }; @@ -109,6 +110,8 @@ ClangTidyOptions::mergeWith(const ClangT Result.HeaderFilterRegex = Other.HeaderFilterRegex; if (Other.AnalyzeTemporaryDtors) Result.AnalyzeTemporaryDtors = Other.AnalyzeTemporaryDtors; + if (Other.User) + Result.User = Other.User; for (const auto &KeyValue : Other.CheckOptions) Result.CheckOptions[KeyValue.first] = KeyValue.second; @@ -118,11 +121,11 @@ ClangTidyOptions::mergeWith(const ClangT FileOptionsProvider::FileOptionsProvider( const ClangTidyGlobalOptions &GlobalOptions, - const ClangTidyOptions &FallbackOptions, + const ClangTidyOptions &DefaultOptions, const ClangTidyOptions &OverrideOptions) - : DefaultOptionsProvider(GlobalOptions, FallbackOptions), + : DefaultOptionsProvider(GlobalOptions, DefaultOptions), OverrideOptions(OverrideOptions) { - CachedOptions[""] = FallbackOptions.mergeWith(OverrideOptions); + CachedOptions[""] = DefaultOptions.mergeWith(OverrideOptions); } static const char ConfigFileName[] = ".clang-tidy"; @@ -177,7 +180,6 @@ llvm::ErrorOr<ClangTidyOptions> FileOptionsProvider::TryReadConfigFile(StringRef Directory) { assert(!Directory.empty()); - ClangTidyOptions Options = DefaultOptionsProvider::getOptions(Directory); if (!llvm::sys::fs::is_directory(Directory)) return make_error_code(llvm::errc::not_a_directory); @@ -201,9 +203,15 @@ FileOptionsProvider::TryReadConfigFile(S // redirection. if ((*Text)->getBuffer().empty()) return make_error_code(llvm::errc::no_such_file_or_directory); - if (std::error_code EC = parseConfiguration((*Text)->getBuffer(), Options)) - return EC; - return Options.mergeWith(OverrideOptions); + llvm::ErrorOr<ClangTidyOptions> ParsedOptions = + parseConfiguration((*Text)->getBuffer()); + if (ParsedOptions) { + ClangTidyOptions Defaults = DefaultOptionsProvider::getOptions(Directory); + // Only use checks from the config file. + Defaults.Checks = None; + return Defaults.mergeWith(*ParsedOptions).mergeWith(OverrideOptions); + } + return ParsedOptions.getError(); } /// \brief Parses -line-filter option and stores it to the \c Options. @@ -214,11 +222,13 @@ std::error_code parseLineFilter(StringRe return Input.error(); } -std::error_code parseConfiguration(StringRef Config, - clang::tidy::ClangTidyOptions &Options) { +llvm::ErrorOr<ClangTidyOptions> parseConfiguration(StringRef Config) { llvm::yaml::Input Input(Config); + ClangTidyOptions Options; Input >> Options; - return Input.error(); + if (Input.error()) + return Input.error(); + return Options; } std::string configurationAsText(const ClangTidyOptions &Options) { Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.h URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.h?rev=218402&r1=218401&r2=218402&view=diff ============================================================================== --- clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.h (original) +++ clang-tools-extra/trunk/clang-tidy/ClangTidyOptions.h Wed Sep 24 13:36:03 2014 @@ -55,6 +55,7 @@ struct ClangTidyOptions { Options.Checks = ""; Options.HeaderFilterRegex = ""; Options.AnalyzeTemporaryDtors = false; + Options.User = llvm::None; return Options; } @@ -72,6 +73,12 @@ struct ClangTidyOptions { /// \brief Turns on temporary destructor-based analysis. llvm::Optional<bool> AnalyzeTemporaryDtors; + /// \brief Specifies the name or e-mail of the user running clang-tidy. + /// + /// This option is used, for example, to place the correct user name in TODO() + /// comments in the relevant check. + llvm::Optional<std::string> User; + typedef std::pair<std::string, std::string> StringPair; typedef std::map<std::string, std::string> OptionMap; @@ -121,19 +128,19 @@ public: /// \param GlobalOptions are just stored and returned to the caller of /// \c getGlobalOptions. /// - /// \param FallbackOptions are used in case there's no corresponding + /// \param DefaultOptions are used for all settings not specified in a /// .clang-tidy file. /// /// If any of the \param OverrideOptions fields are set, they will override /// whatever options are read from the configuration file. FileOptionsProvider(const ClangTidyGlobalOptions &GlobalOptions, - const ClangTidyOptions &FallbackOptions, + const ClangTidyOptions &DefaultOptions, const ClangTidyOptions &OverrideOptions); const ClangTidyOptions &getOptions(llvm::StringRef FileName) override; private: /// \brief Try to read configuration file from \p Directory. If \p Directory - /// is empty, use the fallback value. + /// is empty, use the default value. llvm::ErrorOr<ClangTidyOptions> TryReadConfigFile(llvm::StringRef Directory); llvm::StringMap<ClangTidyOptions> CachedOptions; @@ -144,9 +151,9 @@ private: std::error_code parseLineFilter(llvm::StringRef LineFilter, ClangTidyGlobalOptions &Options); -/// \brief Parses configuration from JSON and stores it to the \p Options. -std::error_code parseConfiguration(llvm::StringRef Config, - ClangTidyOptions &Options); +/// \brief Parses configuration from JSON and returns \c ClangTidyOptions or an +/// error. +llvm::ErrorOr<ClangTidyOptions> parseConfiguration(llvm::StringRef Config); /// \brief Serializes configuration to a YAML-encoded string. std::string configurationAsText(const ClangTidyOptions &Options); Modified: clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.cpp?rev=218402&r1=218401&r2=218402&view=diff ============================================================================== --- clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.cpp (original) +++ clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.cpp Wed Sep 24 13:36:03 2014 @@ -17,8 +17,9 @@ namespace readability { class TodoCommentCheck::TodoCommentHandler : public CommentHandler { public: - explicit TodoCommentHandler(TodoCommentCheck &Check) - : Check(Check), TodoMatch("^// *TODO(\\(.*\\))?:?( )?(.*)$") {} + TodoCommentHandler(TodoCommentCheck &Check, llvm::Optional<std::string> User) + : Check(Check), User(User ? *User : "unknown"), + TodoMatch("^// *TODO(\\(.*\\))?:?( )?(.*)$") {} bool HandleComment(Preprocessor &PP, SourceRange Range) override { StringRef Text = @@ -35,12 +36,6 @@ public: if (!Username.empty()) return false; - // If the username is missing put in the current user's name. Not ideal but - // works for running tidy locally. - // FIXME: Can we get this from a more reliable source? - const char *User = std::getenv("USER"); - if (!User) - User = "unknown"; std::string NewText = ("// TODO(" + Twine(User) + "): " + Comment).str(); Check.diag(Range.getBegin(), "missing username/bug in TODO") @@ -51,14 +46,14 @@ public: private: TodoCommentCheck &Check; + std::string User; llvm::Regex TodoMatch; }; TodoCommentCheck::TodoCommentCheck(StringRef Name, ClangTidyContext *Context) : ClangTidyCheck(Name, Context), - Handler(llvm::make_unique<TodoCommentHandler>(*this)) {} - -TodoCommentCheck::~TodoCommentCheck() {} + Handler(llvm::make_unique<TodoCommentHandler>( + *this, Context->getOptions().User)) {} void TodoCommentCheck::registerPPCallbacks(CompilerInstance &Compiler) { Compiler.getPreprocessor().addCommentHandler(Handler.get()); Modified: clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.h URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.h?rev=218402&r1=218401&r2=218402&view=diff ============================================================================== --- clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.h (original) +++ clang-tools-extra/trunk/clang-tidy/google/TodoCommentCheck.h Wed Sep 24 13:36:03 2014 @@ -22,7 +22,6 @@ namespace readability { class TodoCommentCheck : public ClangTidyCheck { public: TodoCommentCheck(StringRef Name, ClangTidyContext *Context); - ~TodoCommentCheck(); void registerPPCallbacks(CompilerInstance &Compiler) override; private: Modified: clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp?rev=218402&r1=218401&r2=218402&view=diff ============================================================================== --- clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp (original) +++ clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp Wed Sep 24 13:36:03 2014 @@ -17,6 +17,7 @@ #include "../ClangTidy.h" #include "clang/Tooling/CommonOptionsParser.h" +#include "llvm/Support/Process.h" using namespace clang::ast_matchers; using namespace clang::driver; @@ -143,10 +144,14 @@ int clangTidyMain(int argc, const char * return 1; } - ClangTidyOptions FallbackOptions; - FallbackOptions.Checks = DefaultChecks; - FallbackOptions.HeaderFilterRegex = HeaderFilter; - FallbackOptions.AnalyzeTemporaryDtors = AnalyzeTemporaryDtors; + ClangTidyOptions DefaultOptions; + DefaultOptions.Checks = DefaultChecks; + DefaultOptions.HeaderFilterRegex = HeaderFilter; + DefaultOptions.AnalyzeTemporaryDtors = AnalyzeTemporaryDtors; + DefaultOptions.User = llvm::sys::Process::GetEnv("USER"); + // USERNAME is used on Windows. + if (!DefaultOptions.User) + DefaultOptions.User = llvm::sys::Process::GetEnv("USERNAME"); ClangTidyOptions OverrideOptions; if (Checks.getNumOccurrences() > 0) @@ -157,7 +162,7 @@ int clangTidyMain(int argc, const char * OverrideOptions.AnalyzeTemporaryDtors = AnalyzeTemporaryDtors; auto OptionsProvider = llvm::make_unique<FileOptionsProvider>( - GlobalOptions, FallbackOptions, OverrideOptions); + GlobalOptions, DefaultOptions, OverrideOptions); std::string FileName = OptionsParser.getSourcePathList().front(); ClangTidyOptions EffectiveOptions = OptionsProvider->getOptions(FileName); Modified: clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp?rev=218402&r1=218401&r2=218402&view=diff ============================================================================== --- clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp (original) +++ clang-tools-extra/trunk/unittests/clang-tidy/ClangTidyOptionsTest.cpp Wed Sep 24 13:36:03 2014 @@ -55,15 +55,16 @@ TEST(ParseLineFilter, ValidFilter) { } TEST(ParseConfiguration, ValidConfiguration) { - ClangTidyOptions Options; - std::error_code Error = parseConfiguration("Checks: \"-*,misc-*\"\n" - "HeaderFilterRegex: \".*\"\n" - "AnalyzeTemporaryDtors: true\n", - Options); - EXPECT_FALSE(Error); - EXPECT_EQ("-*,misc-*", *Options.Checks); - EXPECT_EQ(".*", *Options.HeaderFilterRegex); - EXPECT_TRUE(*Options.AnalyzeTemporaryDtors); + llvm::ErrorOr<ClangTidyOptions> Options = + parseConfiguration("Checks: \"-*,misc-*\"\n" + "HeaderFilterRegex: \".*\"\n" + "AnalyzeTemporaryDtors: true\n" + "User: some.user"); + EXPECT_TRUE(!!Options); + EXPECT_EQ("-*,misc-*", *Options->Checks); + EXPECT_EQ(".*", *Options->HeaderFilterRegex); + EXPECT_TRUE(*Options->AnalyzeTemporaryDtors); + EXPECT_EQ("some.user", *Options->User); } } // namespace test _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
