diff --git a/src/baseclient.cc b/src/baseclient.cc index 78e38fe..492f501 100644 --- a/src/baseclient.cc +++ b/src/baseclient.cc @@ -130,11 +130,10 @@ void BaseClient::HandleRedirectResponse(std::string& code, std::string& message, } } -Result BaseClient::GetErrorResponse(http::Response resp, - std::string_view resource, - http::Method method, - const std::string& bucket_name, - const std::string& object_name) { +Result BaseClient::GetErrorResponse( + http::Response resp, [[maybe_unused]] std::string_view resource, + http::Method method, const std::string& bucket_name, + const std::string& object_name) { if (!resp.error.empty()) { return tl::make_unexpected(error::Error(resp.error)); } diff --git a/src/client.cc b/src/client.cc index e6bcad8..c1045d7 100644 --- a/src/client.cc +++ b/src/client.cc @@ -269,7 +269,8 @@ struct ScopedRDMARegistration { } // namespace -ListObjectsResult::ListObjectsResult(error::Error err) : failed_(true) { +ListObjectsResult::ListObjectsResult([[maybe_unused]] error::Error err) + : failed_(true) { resp_ = std::make_shared(); itr_ = resp_->contents.end(); } @@ -384,7 +385,7 @@ void ListObjectsResult::Populate() { } } -RemoveObjectsResult::RemoveObjectsResult(error::Error err) { +RemoveObjectsResult::RemoveObjectsResult([[maybe_unused]] error::Error err) { done_ = true; itr_ = resp_.errors.end(); } @@ -1023,7 +1024,7 @@ Result Client::ComposeObject(ComposeObjectArgs args) { amu_args.region = args.region; amu_args.object = args.object; amu_args.upload_id = upload_id; - AbortMultipartUpload(amu_args); + (void)AbortMultipartUpload(amu_args); } return resp; @@ -1586,7 +1587,7 @@ Result Client::PutObject(PutObjectArgs args) { amu_args.region = std::move(args.region); amu_args.object = std::move(args.object); amu_args.upload_id = upload_id; - AbortMultipartUpload(amu_args); + (void)AbortMultipartUpload(amu_args); } return tl::make_unexpected(first_err); } @@ -1611,7 +1612,7 @@ Result Client::PutObject(PutObjectArgs args) { amu_args.region = std::move(args.region); amu_args.object = std::move(args.object); amu_args.upload_id = upload_id; - AbortMultipartUpload(amu_args); + (void)AbortMultipartUpload(amu_args); } if (!cmu_resp) { return tl::make_unexpected(cmu_resp.error()); @@ -1658,7 +1659,7 @@ Result Client::PutObject(PutObjectArgs args) { amu_args.region = std::move(args.region); amu_args.object = std::move(args.object); amu_args.upload_id = upload_id; - AbortMultipartUpload(amu_args); + (void)AbortMultipartUpload(amu_args); } return resp; diff --git a/tests/tests.cc b/tests/tests.cc index 824fc59..7d4de3b 100644 --- a/tests/tests.cc +++ b/tests/tests.cc @@ -798,7 +798,7 @@ class Tests { try { minio::s3::RemoveBucketArgs args; args.bucket = b; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); } catch (...) { } }; @@ -1101,7 +1101,7 @@ class Tests { } catch (const std::runtime_error&) { minio::s3::RemoveBucketArgs args; args.bucket = bucket_name; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); throw; } } @@ -1243,7 +1243,7 @@ class Tests { } catch (const std::runtime_error&) { minio::s3::RemoveBucketArgs args; args.bucket = bucket_name; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); throw; } } @@ -1296,7 +1296,7 @@ class Tests { try { minio::s3::RemoveBucketArgs args; args.bucket = b; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); } catch (...) { } } @@ -1371,7 +1371,7 @@ class Tests { { minio::s3::RemoveBucketArgs args; args.bucket = bucket_name; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); } } @@ -1405,7 +1405,7 @@ class Tests { { minio::s3::RemoveBucketArgs args; args.bucket = bucket_name; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); } } @@ -1439,7 +1439,7 @@ class Tests { { minio::s3::RemoveBucketArgs args; args.bucket = bucket_name; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); } } @@ -1473,7 +1473,7 @@ class Tests { { minio::s3::RemoveBucketArgs args; args.bucket = bucket_name; - client_.RemoveBucket(args); + (void)client_.RemoveBucket(args); } }