Set auto_release's default parameter value

Signed-off-by: Ilho Kim <ilho159.kim@samsung.com>

diff --git a/src/common/parcel/appinfo_parcelable.hh b/src/common/parcel/appinfo_parcelable.hh
index e216ec89c63f7472e6c1649b16f61f09f71995a7..68f60a491b50a6d554d9b56213ce099f53caeb45 100644 (file)
--- a/src/common/parcel/appinfo_parcelable.hh
+++ b/src/common/parcel/appinfo_parcelable.hh
@@ -17,7 +17,7 @@ namespace parcel {
 class EXPORT_API AppInfoParcelable : public AbstractParcelable {
  public:
   AppInfoParcelable();
-  AppInfoParcelable(int ret, std::vector<application_x *>&& app_list, bool auto_release);
+  AppInfoParcelable(int ret, std::vector<application_x *>&& app_list, bool auto_release = true);
   ~AppInfoParcelable();
   const std::vector<application_x *>& GetAppInfo();
   std::vector<application_x *>&& ExtractAppInfo();

diff --git a/src/common/parcel/certinfo_parcelable.hh b/src/common/parcel/certinfo_parcelable.hh
index 8ab08fe35aa827e9588a626f023d69b1a95a85cf..b0dc56702392b62bc73c4441cdce3e047cfb8c8f 100644 (file)
--- a/src/common/parcel/certinfo_parcelable.hh
+++ b/src/common/parcel/certinfo_parcelable.hh
@@ -19,10 +19,10 @@ class EXPORT_API CertInfoParcelable : public AbstractParcelable {
  public:
   CertInfoParcelable();
   CertInfoParcelable(uid_t uid, std::string pkgid);
-  CertInfoParcelable(uid_t uid, pkgmgr_certinfo_x *cert_info, bool auto_release);
-  CertInfoParcelable(int ret, pkgmgr_certinfo_x *cert_info, bool auto_release);
+  CertInfoParcelable(uid_t uid, pkgmgr_certinfo_x *cert_info, bool auto_release = true);
+  CertInfoParcelable(int ret, pkgmgr_certinfo_x *cert_info, bool auto_release = true);
   CertInfoParcelable(uid_t uid,
-      pkgmgr_certinfo_x *cert_info, std::string pkgid, bool auto_release);
+      pkgmgr_certinfo_x *cert_info, std::string pkgid, bool auto_release = true);
   ~CertInfoParcelable();
   const pkgmgr_certinfo_x *GetCertInfo();
   pkgmgr_certinfo_x *ExtractCertInfo();

diff --git a/src/common/parcel/depinfo_parcelable.hh b/src/common/parcel/depinfo_parcelable.hh
index 0dcce2266495bb1c3faad9eebbc7951ca13af821..c78e77876245c0e991b97326a1ba0cea195660f7 100644 (file)
--- a/src/common/parcel/depinfo_parcelable.hh
+++ b/src/common/parcel/depinfo_parcelable.hh
@@ -17,7 +17,7 @@ namespace parcel {
 class EXPORT_API DepInfoParcelable : public AbstractParcelable {
  public:
   DepInfoParcelable();
-  DepInfoParcelable(int ret, std::vector<dependency_x *>&& dependency_list, bool auto_release);
+  DepInfoParcelable(int ret, std::vector<dependency_x *>&& dependency_list, bool auto_release = true);
   DepInfoParcelable(const std::string& pkgid);
   ~DepInfoParcelable();
   const std::vector<dependency_x *>& GetDependencyInfo();

diff --git a/src/common/parcel/filter_parcelable.hh b/src/common/parcel/filter_parcelable.hh
index cc3d210389d52e2d87326d682155ae94df189c39..9615fc72761222344d24d5a0f0190b38165bb839 100644 (file)
--- a/src/common/parcel/filter_parcelable.hh
+++ b/src/common/parcel/filter_parcelable.hh
@@ -17,7 +17,7 @@ namespace parcel {
 class EXPORT_API FilterParcelable : public AbstractParcelable {
  public:
   FilterParcelable();
-  FilterParcelable(uid_t uid, pkgmgrinfo_filter_x *filter, int flag, bool auto_release);
+  FilterParcelable(uid_t uid, pkgmgrinfo_filter_x *filter, int flag, bool auto_release = true);
   ~FilterParcelable();
   const pkgmgrinfo_filter_x* GetFilter();
   int GetFlag();

diff --git a/src/common/parcel/pkginfo_parcelable.hh b/src/common/parcel/pkginfo_parcelable.hh
index 24b1d8254da997fa58f938dab7043d34215d6cdd..1c0d8c316c530c17ed8f6678098ff8a16a24cffb 100644 (file)
--- a/src/common/parcel/pkginfo_parcelable.hh
+++ b/src/common/parcel/pkginfo_parcelable.hh
@@ -18,8 +18,8 @@ namespace parcel {
 class EXPORT_API PkgInfoParcelable : public AbstractParcelable {
  public:
   PkgInfoParcelable();
-  PkgInfoParcelable(uid_t uid, std::vector<package_x *>&& pkg_list, WriteType write_type, bool auto_release);
-  PkgInfoParcelable(int ret, std::vector<package_x *>&& pkg_list, bool auto_release);
+  PkgInfoParcelable(uid_t uid, std::vector<package_x *>&& pkg_list, WriteType write_type, bool auto_release = true);
+  PkgInfoParcelable(int ret, std::vector<package_x *>&& pkg_list, bool auto_release = true);
   ~PkgInfoParcelable();
   const std::vector<package_x *>& GetPkgInfo();
   std::vector<package_x *>&& ExtractPkgInfo();

diff --git a/src/common/request_handler/get_appinfo_request_handler.cc b/src/common/request_handler/get_appinfo_request_handler.cc
index 6987a39a79e431c296996d34761c6c5dcc1086e9..049f0decf372c171b59553bb5ded6e98c941397c 100644 (file)
--- a/src/common/request_handler/get_appinfo_request_handler.cc
+++ b/src/common/request_handler/get_appinfo_request_handler.cc
@@ -25,7 +25,7 @@ bool GetAppinfoRequestHandler::HandleRequest(unsigned char* data, int size,
       abstract_parcel->GetType() != ParcelableType::Filter) {
     _LOGE("Invalid parcel or type");
     result_ = std::make_shared<AppInfoParcelable>(
-        PMINFO_R_ERROR, std::vector<application_x*>{}, true);
+        PMINFO_R_ERROR, std::vector<application_x*>{});
     return false;
   }
 
@@ -33,7 +33,7 @@ bool GetAppinfoRequestHandler::HandleRequest(unsigned char* data, int size,
   if (parcel == nullptr) {
     _LOGE("Parcel is empty");
     result_ = std::make_shared<AppInfoParcelable>(
-        PMINFO_R_ERROR, std::vector<application_x*>{}, true);
+        PMINFO_R_ERROR, std::vector<application_x*>{});
     return false;
   }
 
@@ -44,7 +44,7 @@ bool GetAppinfoRequestHandler::HandleRequest(unsigned char* data, int size,
   if (ret < 0)
     _LOGE("Failed to execute");
 
-  result_ = std::make_shared<AppInfoParcelable>(ret, db.GetAppHandle(), true);
+  result_ = std::make_shared<AppInfoParcelable>(ret, db.GetAppHandle());
 
   return true;
 }

diff --git a/src/common/request_handler/get_cert_request_handler.cc b/src/common/request_handler/get_cert_request_handler.cc
index b12f823e14f5bd84ef489c5cb63cd698fdc30cca..ee3147291013f38f86c37082e2f810e97e9c82da 100644 (file)
--- a/src/common/request_handler/get_cert_request_handler.cc
+++ b/src/common/request_handler/get_cert_request_handler.cc
@@ -24,7 +24,7 @@ bool GetCertRequestHandler::HandleRequest(unsigned char* data, int size,
       abstract_parcel->GetType() != ParcelableType::CertInfo) {
     _LOGE("Invalid parcel or type");
     result_ = std::make_shared<CertInfoParcelable>(
-        PMINFO_R_ERROR, nullptr, true);
+        PMINFO_R_ERROR, nullptr);
     return false;
   }
 
@@ -32,7 +32,7 @@ bool GetCertRequestHandler::HandleRequest(unsigned char* data, int size,
   if (parcel == nullptr) {
     _LOGE("Parcel is empty");
     result_ = std::make_shared<CertInfoParcelable>(
-        PMINFO_R_ERROR, nullptr, true);
+        PMINFO_R_ERROR, nullptr);
     return false;
   }
 
@@ -42,7 +42,7 @@ bool GetCertRequestHandler::HandleRequest(unsigned char* data, int size,
 
   int ret = db.Execute();
 
-  result_ = std::make_shared<CertInfoParcelable>(ret, db.GetCertHandle(), true);
+  result_ = std::make_shared<CertInfoParcelable>(ret, db.GetCertHandle());
 
   return true;
 }

diff --git a/src/common/request_handler/get_depinfo_request_handler.cc b/src/common/request_handler/get_depinfo_request_handler.cc
index 35587b8133a908b943e20fe0e0a30f7d5c9c47f3..c9c4aba215675adf8ed340c3c24fdd78e87dc7a9 100644 (file)
--- a/src/common/request_handler/get_depinfo_request_handler.cc
+++ b/src/common/request_handler/get_depinfo_request_handler.cc
@@ -25,7 +25,7 @@ bool GetDepinfoRequestHandler::HandleRequest(unsigned char* data, int size,
       abstract_parcel->GetType() != ParcelableType::DepInfo) {
     _LOGE("Invalid parcel or type");
     result_ = std::make_shared<DepInfoParcelable>(
-        PMINFO_R_ERROR, std::vector<dependency_x*>{}, true);
+        PMINFO_R_ERROR, std::vector<dependency_x*>{});
     return false;
   }
 
@@ -33,7 +33,7 @@ bool GetDepinfoRequestHandler::HandleRequest(unsigned char* data, int size,
   if (parcel == nullptr) {
     _LOGE("Parcel is empty");
     result_ = std::make_shared<DepInfoParcelable>(
-        PMINFO_R_ERROR, std::vector<dependency_x*>{}, true);
+        PMINFO_R_ERROR, std::vector<dependency_x*>{});
     return false;
   }
 
@@ -41,7 +41,7 @@ bool GetDepinfoRequestHandler::HandleRequest(unsigned char* data, int size,
   db.SetPkgID(parcel->GetPkgID());
   int ret = db.Execute();
 
-  result_ = std::make_shared<DepInfoParcelable>(ret, db.GetDependencyList(), true);
+  result_ = std::make_shared<DepInfoParcelable>(ret, db.GetDependencyList());
   return true;
 }
 

diff --git a/src/common/request_handler/get_pkginfo_request_handler.cc b/src/common/request_handler/get_pkginfo_request_handler.cc
index 306330aa73772ab6701d4b20becb4b319bbaf32b..0307c1163dcbdb1cb40b63c5c15ff96ceb4820b6 100644 (file)
--- a/src/common/request_handler/get_pkginfo_request_handler.cc
+++ b/src/common/request_handler/get_pkginfo_request_handler.cc
@@ -25,7 +25,7 @@ bool GetPkginfoRequestHandler::HandleRequest(unsigned char* data, int size,
       abstract_parcel->GetType() != ParcelableType::Filter) {
     _LOGE("Invalid parcel or type");
     result_ = std::make_shared<PkgInfoParcelable>(
-        PMINFO_R_ERROR, std::vector<package_x*>{}, true);
+        PMINFO_R_ERROR, std::vector<package_x*>{});
     return false;
   }
 
@@ -33,7 +33,7 @@ bool GetPkginfoRequestHandler::HandleRequest(unsigned char* data, int size,
   if (parcel == nullptr) {
     _LOGE("Parcel is empty");
     result_ = std::make_shared<PkgInfoParcelable>(
-        PMINFO_R_ERROR, std::vector<package_x*>{}, true);
+        PMINFO_R_ERROR, std::vector<package_x*>{});
     return false;
   }
 
@@ -42,7 +42,7 @@ bool GetPkginfoRequestHandler::HandleRequest(unsigned char* data, int size,
   db.SetFilter(const_cast<pkgmgrinfo_filter_x*>(parcel->GetFilter()));
   int ret = db.Execute();
 
-  result_ = std::make_shared<PkgInfoParcelable>(ret, db.GetPkgHandle(), true);
+  result_ = std::make_shared<PkgInfoParcelable>(ret, db.GetPkgHandle());
 
   return true;
 }

diff --git a/test/unit_tests/test_parcel.cc b/test/unit_tests/test_parcel.cc
index ccbc7561712efd904ac10bc47c82632eb42c7249..d6be955fd4b626e916d44700cd2f0c186cf9077b 100644 (file)
--- a/test/unit_tests/test_parcel.cc
+++ b/test/unit_tests/test_parcel.cc
@@ -107,7 +107,7 @@ TEST_F(ParcelTest, AppInfoParcelable) {
 
   origin_applications.emplace_back(GetTestApplication("test_appid1"));
   origin_applications.emplace_back(GetTestApplication("test_appid2"));
-  pp::AppInfoParcelable origin_parcelable(0, std::move(origin_applications), true);
+  pp::AppInfoParcelable origin_parcelable(0, std::move(origin_applications));
   pp::AppInfoParcelable new_parcelable;
   parcel.WriteParcelable(origin_parcelable);
   parcel.ReadParcelable(&new_parcelable);
@@ -122,7 +122,7 @@ TEST_F(ParcelTest, PkgInfoParcelable) {
 
   origin_packages.emplace_back(GetTestPackage("test_pkgid1"));
   origin_packages.emplace_back(GetTestPackage("test_pkgid2"));
-  pp::PkgInfoParcelable origin_parcelable(0, std::move(origin_packages), true);
+  pp::PkgInfoParcelable origin_parcelable(0, std::move(origin_packages));
   pp::PkgInfoParcelable new_parcelable;
   parcel.WriteParcelable(origin_parcelable);
   parcel.ReadParcelable(&new_parcelable);
@@ -136,7 +136,7 @@ TEST_F(ParcelTest, FilterParcelable) {
   pkgmgrinfo_filter_x *filter = GetTestFilter();
   int flag = 0b101010;
 
-  pp::FilterParcelable origin_parcelable(0, filter, flag, true);
+  pp::FilterParcelable origin_parcelable(0, filter, flag);
   pp::FilterParcelable new_parcelable;
   parcel.WriteParcelable(origin_parcelable);
   parcel.ReadParcelable(&new_parcelable);
@@ -151,7 +151,7 @@ TEST_F(ParcelTest, CertInfoParcelable) {
   pkgmgr_certinfo_x *certinfo = GetTestCertInfo();
   std::string test_str = "test_str";
 
-  pp::CertInfoParcelable origin_parcelable(0, certinfo, test_str, true);
+  pp::CertInfoParcelable origin_parcelable(0, certinfo, test_str);
   pp::CertInfoParcelable new_parcelable;
   parcel.WriteParcelable(origin_parcelable);
   parcel.ReadParcelable(&new_parcelable);