diff options
| author | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2017-04-21 17:19:29 (GMT) |
|---|---|---|
| committer | Daniel Pfeifer <daniel@pfeifer-mail.de> | 2017-04-26 06:59:56 (GMT) |
| commit | ba8571ff2deb3a29bb980e109abef0e7290de56a (patch) | |
| tree | 4c43de0a1a78472e35ba6cac1259b62087080dcd /Source/CPack | |
| parent | 8985c1dfd79804d6318e6dae6e47597a7b71dcd4 (diff) | |
| download | CMake-ba8571ff2deb3a29bb980e109abef0e7290de56a.zip CMake-ba8571ff2deb3a29bb980e109abef0e7290de56a.tar.gz CMake-ba8571ff2deb3a29bb980e109abef0e7290de56a.tar.bz2 | |
clang-tidy: use operators for string comparison
Diffstat (limited to 'Source/CPack')
| -rw-r--r-- | Source/CPack/IFW/cmCPackIFWInstaller.cxx | 5 | ||||
| -rw-r--r-- | Source/CPack/IFW/cmCPackIFWPackage.cxx | 6 |
2 files changed, 5 insertions, 6 deletions
diff --git a/Source/CPack/IFW/cmCPackIFWInstaller.cxx b/Source/CPack/IFW/cmCPackIFWInstaller.cxx index 57b47f1..664048d 100644 --- a/Source/CPack/IFW/cmCPackIFWInstaller.cxx +++ b/Source/CPack/IFW/cmCPackIFWInstaller.cxx @@ -171,9 +171,8 @@ void cmCPackIFWInstaller::ConfigureFromOptions() // WizardStyle if (const char* option = GetOption("CPACK_IFW_PACKAGE_WIZARD_STYLE")) { - if (WizardStyle.compare("Modern") == 0 && - WizardStyle.compare("Aero") == 0 && WizardStyle.compare("Mac") == 0 && - WizardStyle.compare("Classic") == 0) { + if (WizardStyle == "Modern" && WizardStyle == "Aero" && + WizardStyle == "Mac" && WizardStyle == "Classic") { cmCPackLogger( cmCPackLog::LOG_WARNING, "Option CPACK_IFW_PACKAGE_WIZARD_STYLE has unknown value \"" diff --git a/Source/CPack/IFW/cmCPackIFWPackage.cxx b/Source/CPack/IFW/cmCPackIFWPackage.cxx index 99e8b9e..eda383f 100644 --- a/Source/CPack/IFW/cmCPackIFWPackage.cxx +++ b/Source/CPack/IFW/cmCPackIFWPackage.cxx @@ -514,11 +514,11 @@ int cmCPackIFWPackage::ConfigureFromPrefix(const std::string& prefix) Default.clear(); } else if (const char* value = GetOption(option)) { std::string lowerValue = cmsys::SystemTools::LowerCase(value); - if (lowerValue.compare("true") == 0) { + if (lowerValue == "true") { Default = "true"; - } else if (lowerValue.compare("false") == 0) { + } else if (lowerValue == "false") { Default = "false"; - } else if (lowerValue.compare("script") == 0) { + } else if (lowerValue == "script") { Default = "script"; } else { Default = value; |