resolve compiler warnings (#1246)

* resolve compiler warnings

- check `WSAStartup` return.
- `const` is not suitable for `std::move`.

* resolve compiler warnings

- bool startup => bool is_valid_.
- remove `const` not removed.
This commit is contained in:
greenfish 2022-04-15 00:46:10 +09:00 committed by GitHub
parent cb41947eb4
commit 5d87cc0558
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -974,7 +974,7 @@ public:
void stop(); void stop();
void set_hostname_addr_map(const std::map<std::string, std::string> addr_map); void set_hostname_addr_map(std::map<std::string, std::string> addr_map);
void set_default_headers(Headers headers); void set_default_headers(Headers headers);
@ -1309,7 +1309,7 @@ public:
void stop(); void stop();
void set_hostname_addr_map(const std::map<std::string, std::string> addr_map); void set_hostname_addr_map(std::map<std::string, std::string> addr_map);
void set_default_headers(Headers headers); void set_default_headers(Headers headers);
@ -4231,10 +4231,12 @@ class WSInit {
public: public:
WSInit() { WSInit() {
WSADATA wsaData; WSADATA wsaData;
WSAStartup(0x0002, &wsaData); if (WSAStartup(0x0002, &wsaData) == 0) is_valid_ = true;
} }
~WSInit() { WSACleanup(); } ~WSInit() { if (is_valid_) WSACleanup(); }
bool is_valid_ = false;
}; };
static WSInit wsinit_; static WSInit wsinit_;
@ -6969,7 +6971,7 @@ inline void ClientImpl::set_follow_location(bool on) { follow_location_ = on; }
inline void ClientImpl::set_url_encode(bool on) { url_encode_ = on; } inline void ClientImpl::set_url_encode(bool on) { url_encode_ = on; }
inline void ClientImpl::set_hostname_addr_map( inline void ClientImpl::set_hostname_addr_map(
const std::map<std::string, std::string> addr_map) { std::map<std::string, std::string> addr_map) {
addr_map_ = std::move(addr_map); addr_map_ = std::move(addr_map);
} }
@ -8095,7 +8097,7 @@ inline size_t Client::is_socket_open() const { return cli_->is_socket_open(); }
inline void Client::stop() { cli_->stop(); } inline void Client::stop() { cli_->stop(); }
inline void Client::set_hostname_addr_map( inline void Client::set_hostname_addr_map(
const std::map<std::string, std::string> addr_map) { std::map<std::string, std::string> addr_map) {
cli_->set_hostname_addr_map(std::move(addr_map)); cli_->set_hostname_addr_map(std::move(addr_map));
} }