change client_ to shared_ptr type

aliyun · May 25, 2019 · ee12a0b · ee12a0b
1 parent 5b71310
commit ee12a0b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/sdk/include/alibabacloud/oss/OssClient.h b/sdk/include/alibabacloud/oss/OssClient.h
@@ -239,7 +239,7 @@ namespace OSS
         void DisableRequest();
         void EnableRequest();
     private:
-        OssClientImpl *client_;
+        std::shared_ptr<OssClientImpl> client_;
     };
 }
 }
diff --git a/sdk/src/OssClient.cc b/sdk/src/OssClient.cc
@@ -90,16 +90,12 @@ OssClient::OssClient(const std::string &endpoint, const Credentials &credentials
 }
 
 OssClient::OssClient(const std::string &endpoint, const std::shared_ptr<CredentialsProvider>& credentialsProvider, const ClientConfiguration & configuration) :
-    client_(new OssClientImpl(endpoint, credentialsProvider, configuration))
+    client_(std::make_shared<OssClientImpl>(endpoint, credentialsProvider, configuration))
 {
 }
 
 OssClient::~OssClient()
 {
-    if (client_) {
-        delete client_;
-    }
-    client_ = nullptr;
 }
 
 ListBucketsOutcome OssClient::ListBuckets() const