From 1d593585e35d4410368f9149352431dedae26107 Mon Sep 17 00:00:00 2001 From: Gabriele Contini <contini.mailing@gmail.com> Date: 周六, 14 12月 2019 19:17:07 +0800 Subject: [PATCH] codacy warnings --- src/library/locate/LocatorFactory.cpp | 25 +++++++++++++++---------- 1 files changed, 15 insertions(+), 10 deletions(-) diff --git a/src/library/locate/LocatorFactory.cpp b/src/library/locate/LocatorFactory.cpp index 667a387..c60d4d5 100644 --- a/src/library/locate/LocatorFactory.cpp +++ b/src/library/locate/LocatorFactory.cpp @@ -5,8 +5,6 @@ * Author: Gabriele Contini */ -#include "licensecc_properties.h" - #include "LocatorStrategy.hpp" #include "LocatorFactory.hpp" #include "ApplicationFolder.hpp" @@ -16,18 +14,25 @@ namespace license { namespace locate { +static std::vector<std::unique_ptr<LocatorStrategy>> extra_strategies; +bool LocatorFactory::find_license_near_moduleb = FIND_LICENSE_NEAR_MODULE; +bool LocatorFactory::find_license_with_env_varb = FIND_LICENSE_WITH_ENV_VAR; FUNCTION_RETURN LocatorFactory::get_active_strategies(std::vector<std::unique_ptr<LocatorStrategy>> &strategies, const LicenseLocation *locationHint) { -#if (FIND_LICENSE_NEAR_MODULE) - strategies.push_back(std::unique_ptr<LocatorStrategy>((LocatorStrategy *)new ApplicationFolder())); -#endif -#if (FIND_LICENSE_WITH_ENV_VAR) - strategies.push_back(std::unique_ptr<LocatorStrategy>((LocatorStrategy *)new EnvironmentVarLocation())); - strategies.push_back(std::unique_ptr<LocatorStrategy>((LocatorStrategy *)new EnvironmentVarData())); -#endif + if (find_license_near_moduleb) { + strategies.push_back( + std::unique_ptr<LocatorStrategy>(dynamic_cast<LocatorStrategy *>(new ApplicationFolder()))); + } + if (find_license_with_env_varb) { + strategies.push_back( + std::unique_ptr<LocatorStrategy>(dynamic_cast<LocatorStrategy *>(new EnvironmentVarLocation()))); + strategies.push_back( + std::unique_ptr<LocatorStrategy>(dynamic_cast<LocatorStrategy *>(new EnvironmentVarData()))); + } if (locationHint != nullptr) { - strategies.push_back(std::unique_ptr<LocatorStrategy>((LocatorStrategy *)new ExternalDefinition(locationHint))); + strategies.push_back( + std::unique_ptr<LocatorStrategy>(dynamic_cast<LocatorStrategy *>(new ExternalDefinition(locationHint)))); } return strategies.size() > 0 ? FUNC_RET_OK : FUNC_RET_NOT_AVAIL; } -- Gitblit v1.9.1