diff options
author | Brad King <brad.king@kitware.com> | 2018-03-16 12:21:29 -0400 |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2018-03-16 12:21:29 -0400 |
commit | 1d00ed7cf7ffd5c51dfcf77075589ab0e30d29a3 (patch) | |
tree | febf97882d269d90614e1bf098a0eca9769fa3b6 /Source | |
parent | 0df559832bdc629df333b57c798d42e77716a94b (diff) | |
download | cmake-1d00ed7cf7ffd5c51dfcf77075589ab0e30d29a3.tar.gz |
cmPolicies: Drop unnecessary check from ApplyPolicyVersion
This internal API is never called with `version_min` empty.
Diffstat (limited to 'Source')
-rw-r--r-- | Source/cmPolicies.cxx | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/Source/cmPolicies.cxx b/Source/cmPolicies.cxx index 7e3bd78dec..adb6e78852 100644 --- a/Source/cmPolicies.cxx +++ b/Source/cmPolicies.cxx @@ -156,22 +156,16 @@ static bool GetPolicyDefault(cmMakefile* mf, std::string const& policy, bool cmPolicies::ApplyPolicyVersion(cmMakefile* mf, std::string const& version_min) { - std::string ver = "2.4.0"; - - if (!version_min.empty()) { - ver = version_min; - } - unsigned int majorVer = 2; unsigned int minorVer = 0; unsigned int patchVer = 0; unsigned int tweakVer = 0; // parse the string - if (sscanf(ver.c_str(), "%u.%u.%u.%u", &majorVer, &minorVer, &patchVer, - &tweakVer) < 2) { + if (sscanf(version_min.c_str(), "%u.%u.%u.%u", &majorVer, &minorVer, + &patchVer, &tweakVer) < 2) { std::ostringstream e; - e << "Invalid policy version value \"" << ver << "\". " + e << "Invalid policy version value \"" << version_min << "\". " << "A numeric major.minor[.patch[.tweak]] must be given."; mf->IssueMessage(cmake::FATAL_ERROR, e.str()); return false; |