[core] Fix bugprone-exception-escape errors
As reported by clang-tidy-8.
This commit is contained in:
@@ -266,7 +266,10 @@ HTTPRequest::HTTPRequest(HTTPFileSource::Impl* context_, Resource resource_, Fil
|
||||
}
|
||||
|
||||
HTTPRequest::~HTTPRequest() {
|
||||
handleError(curl_multi_remove_handle(context->multi, handle));
|
||||
if (curl_multi_remove_handle(context->multi, handle) != CURLM_OK) {
|
||||
mbgl::Log::Error(mbgl::Event::HttpRequest, "Error removing curl multi handle");
|
||||
}
|
||||
|
||||
context->returnHandle(handle);
|
||||
handle = nullptr;
|
||||
|
||||
|
||||
@@ -29,6 +29,7 @@
|
||||
#include <mbgl/style/style.hpp>
|
||||
#include <mbgl/util/async_request.hpp>
|
||||
#include <mbgl/util/exception.hpp>
|
||||
#include <mbgl/util/logging.hpp>
|
||||
#include <mbgl/util/premultiply.hpp>
|
||||
|
||||
#include <unistd.h>
|
||||
@@ -1428,7 +1429,11 @@ NodeMap::NodeMap(v8::Local<v8::Object> options)
|
||||
}
|
||||
|
||||
NodeMap::~NodeMap() {
|
||||
if (map) release();
|
||||
try {
|
||||
if (map) release();
|
||||
} catch (...) {
|
||||
mbgl::Log::Error(mbgl::Event::General, "Error release the map object when destroying NodeMap");
|
||||
}
|
||||
}
|
||||
|
||||
std::unique_ptr<mbgl::AsyncRequest> NodeFileSource::request(const mbgl::Resource& resource, mbgl::FileSource::Callback callback_) {
|
||||
|
||||
Reference in New Issue
Block a user