diff options
author | Brad King <brad.king@kitware.com> | 2019-01-16 14:52:22 (GMT) |
---|---|---|
committer | Kitware Robot <kwrobot@kitware.com> | 2019-01-16 14:52:45 (GMT) |
commit | bd54cc774a5b14720a9e741fc40192ea11106443 (patch) | |
tree | f8d0bab5ca429938678e74af746ee9ab0ccdd778 /Source/cmFileCommand.cxx | |
parent | e2c39e05732409d60e7921b45a70dd8b9aa24b5a (diff) | |
parent | b056bc34258267c4ae59c72e3777d0a0f0b5a1b1 (diff) | |
download | CMake-bd54cc774a5b14720a9e741fc40192ea11106443.zip CMake-bd54cc774a5b14720a9e741fc40192ea11106443.tar.gz CMake-bd54cc774a5b14720a9e741fc40192ea11106443.tar.bz2 |
Merge topic 'semi-warnings'
b056bc3425 Fix most clang -Wextra-semi-stmt warnings in C++ files
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2795
Diffstat (limited to 'Source/cmFileCommand.cxx')
-rw-r--r-- | Source/cmFileCommand.cxx | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/Source/cmFileCommand.cxx b/Source/cmFileCommand.cxx index 5c93068..db2fde8 100644 --- a/Source/cmFileCommand.cxx +++ b/Source/cmFileCommand.cxx @@ -2700,12 +2700,14 @@ private: #endif #define check_curl_result(result, errstr) \ - if (result != CURLE_OK) { \ - std::string e(errstr); \ - e += ::curl_easy_strerror(result); \ - this->SetError(e); \ - return false; \ - } + do { \ + if (result != CURLE_OK) { \ + std::string e(errstr); \ + e += ::curl_easy_strerror(result); \ + this->SetError(e); \ + return false; \ + } \ + } while (false) bool cmFileCommand::HandleDownloadCommand(std::vector<std::string> const& args) { |