Added: smart pointers in ParameterMgrPlatformConnector issue intel#311

vlmarkov · Jan 28, 2019 · 432af36 · 432af36
1 parent 4842b32
commit 432af36
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/parameter/ParameterMgrPlatformConnector.cpp b/parameter/ParameterMgrPlatformConnector.cpp
@@ -37,16 +37,15 @@ using std::string;
 // Construction
 CParameterMgrPlatformConnector::CParameterMgrPlatformConnector(
     const string &strConfigurationFilePath)
-    : _pParameterMgrLogger(new CParameterMgrLogger<CParameterMgrPlatformConnector>(*this)),
-      _pParameterMgr(new CParameterMgr(strConfigurationFilePath, *_pParameterMgrLogger)),
+    : _pParameterMgrLogger(utility::make_unique<CParameterMgrLogger<CParameterMgrPlatformConnector>>(*this)),
+      _pParameterMgr(utility::make_unique<CParameterMgr>(strConfigurationFilePath, *_pParameterMgrLogger)),
       _bStarted(false), _pLogger(nullptr)
 {
 }
 
 CParameterMgrPlatformConnector::~CParameterMgrPlatformConnector()
 {
-    delete _pParameterMgr;
-    delete _pParameterMgrLogger;
+    ;
 }
 
 // Selection Criteria interface. Beware returned objects are lent, clients shall not delete them!

diff --git a/parameter/include/ParameterMgrPlatformConnector.h b/parameter/include/ParameterMgrPlatformConnector.h
@@ -36,6 +36,7 @@
 #include "ParameterHandle.h"
 #include "ElementHandle.h"
 #include "ParameterMgrLoggerForward.h"
+#include "Memory.hpp"
 
 class CParameterMgr;
 
@@ -196,9 +197,9 @@ class PARAMETER_EXPORT CParameterMgrPlatformConnector
 
 protected:
     // Private logging
-    CParameterMgrLogger<CParameterMgrPlatformConnector> *_pParameterMgrLogger;
+    std::unique_ptr<CParameterMgrLogger<CParameterMgrPlatformConnector>> _pParameterMgrLogger;
     // Implementation
-    CParameterMgr *_pParameterMgr;
+    std::unique_ptr<CParameterMgr> _pParameterMgr;
     // State
     bool _bStarted;
     // Logging