Remove IFactory innerclass from parcelable classes

Signed-off-by: Junghyun Yeon <jungh.yeon@samsung.com>

diff --git a/src/common/parcel/abstract_parcelable.hh b/src/common/parcel/abstract_parcelable.hh
index e8ae023..dcd3b15 100644 (file)
--- a/src/common/parcel/abstract_parcelable.hh
+++ b/src/common/parcel/abstract_parcelable.hh
@@ -28,11 +28,6 @@ enum ParcelableType {
 
 class EXPORT_API AbstractParcelable : public tizen_base::Parcelable {
  public:
-  class EXPORT_API IFactory {
-   public:
-    virtual std::unique_ptr<AbstractParcelable> CreateParcel(std::vector<unsigned char> data) = 0;
-  };
-
   AbstractParcelable() : uid_(0), type_(ParcelableType::Unknown) {}
   AbstractParcelable(uid_t uid, ParcelableType type) : uid_(uid), type_(type) {}
 

diff --git a/src/common/parcel/appinfo_parcelable.cc b/src/common/parcel/appinfo_parcelable.cc
index 9258f1e..84ed391 100644 (file)
--- a/src/common/parcel/appinfo_parcelable.cc
+++ b/src/common/parcel/appinfo_parcelable.cc
@@ -551,9 +551,5 @@ void AppInfoParcelable::ReadFromParcel(tizen_base::Parcel* parcel) {
   }
 }
 
-std::unique_ptr<AbstractParcelable> AppInfoParcelable::Factory::CreateParcel() {
-  return nullptr;
-}
-
 }  // namespace parcel
 }  // namespace pkgmgr_common

diff --git a/src/common/parcel/appinfo_parcelable.hh b/src/common/parcel/appinfo_parcelable.hh
index 26a3d51..4f8d1be 100644 (file)
--- a/src/common/parcel/appinfo_parcelable.hh
+++ b/src/common/parcel/appinfo_parcelable.hh
@@ -16,11 +16,6 @@ namespace parcel {
 
 class EXPORT_API AppInfoParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   AppInfoParcelable();
   AppInfoParcelable(std::vector<application_x *>&& app_list);
   const std::vector<application_x *>& GetAppInfo();

diff --git a/src/common/parcel/certinfo_parcelable.cc b/src/common/parcel/certinfo_parcelable.cc
index 619508b..03398a0 100644 (file)
--- a/src/common/parcel/certinfo_parcelable.cc
+++ b/src/common/parcel/certinfo_parcelable.cc
@@ -106,10 +106,5 @@ void CertInfoParcelable::ReadPkgId(tizen_base::Parcel* parcel) {
   pkgid_ = parcel->ReadString();
 }
 
-std::unique_ptr<AbstractParcelable>
-CertInfoParcelable::Factory::CreateParcel() {
-  return nullptr;
-}
-
 }  // namespace parcel
 }  // namespace pkgmgr_common

diff --git a/src/common/parcel/certinfo_parcelable.hh b/src/common/parcel/certinfo_parcelable.hh
index a56bc02..b1bb0c3 100644 (file)
--- a/src/common/parcel/certinfo_parcelable.hh
+++ b/src/common/parcel/certinfo_parcelable.hh
@@ -17,11 +17,6 @@ namespace parcel {
 
 class EXPORT_API CertInfoParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   CertInfoParcelable();
   CertInfoParcelable(uid_t uid, std::string pkgid);
   CertInfoParcelable(uid_t uid, pkgmgr_certinfo_x *cert_info);

diff --git a/src/common/parcel/depinfo_parcelable.cc b/src/common/parcel/depinfo_parcelable.cc
index dc234a8..1da6e52 100644 (file)
--- a/src/common/parcel/depinfo_parcelable.cc
+++ b/src/common/parcel/depinfo_parcelable.cc
@@ -87,9 +87,5 @@ dependency_x *DepInfoParcelable::ReadDependency(tizen_base::Parcel* parcel) {
   return dep;
 }
 
-std::unique_ptr<AbstractParcelable> DepInfoParcelable::Factory::CreateParcel() {
-  return nullptr;
-}
-
 }  // namespace parcel
 }  // namespace pkgmgr_common

diff --git a/src/common/parcel/depinfo_parcelable.hh b/src/common/parcel/depinfo_parcelable.hh
index a14a153..d219bb4 100644 (file)
--- a/src/common/parcel/depinfo_parcelable.hh
+++ b/src/common/parcel/depinfo_parcelable.hh
@@ -16,11 +16,6 @@ namespace parcel {
 
 class EXPORT_API DepInfoParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   DepInfoParcelable();
   DepInfoParcelable(std::vector<dependency_x *>&& dependency_list);
   DepInfoParcelable(const std::string& pkgid);

diff --git a/src/common/parcel/filter_parcelable.cc b/src/common/parcel/filter_parcelable.cc
index ce00ccb..724761e 100644 (file)
--- a/src/common/parcel/filter_parcelable.cc
+++ b/src/common/parcel/filter_parcelable.cc
@@ -135,9 +135,5 @@ pkgmgrinfo_metadata_node_x *FilterParcelable::ReadPkgmgrInfoMetadataNode(
   return node;
 }
 
-std::unique_ptr<AbstractParcelable> FilterParcelable::Factory::CreateParcel() {
-  return nullptr;
-}
-
 }  // namespace parcel
 }  // namespace pkgmgr_common

diff --git a/src/common/parcel/filter_parcelable.hh b/src/common/parcel/filter_parcelable.hh
index f5719d4..0be9227 100644 (file)
--- a/src/common/parcel/filter_parcelable.hh
+++ b/src/common/parcel/filter_parcelable.hh
@@ -16,11 +16,6 @@ namespace parcel {
 
 class EXPORT_API FilterParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   FilterParcelable();
   FilterParcelable(uid_t uid, pkgmgrinfo_filter_x *filter, int flag);
   const pkgmgrinfo_filter_x* GetFilter();

diff --git a/src/common/parcel/pkginfo_parcelable.cc b/src/common/parcel/pkginfo_parcelable.cc
index 2705528..1f179e0 100644 (file)
--- a/src/common/parcel/pkginfo_parcelable.cc
+++ b/src/common/parcel/pkginfo_parcelable.cc
@@ -477,9 +477,5 @@ void PkgInfoParcelable::ReadFromParcel(tizen_base::Parcel* parcel) {
   }
 }
 
-std::unique_ptr<AbstractParcelable> PkgInfoParcelable::Factory::CreateParcel() {
-  return nullptr;
-}
-
 }  // namespace parcel
 }  // namespace pkgmgr_common

diff --git a/src/common/parcel/pkginfo_parcelable.hh b/src/common/parcel/pkginfo_parcelable.hh
index 0138677..f5f76be 100644 (file)
--- a/src/common/parcel/pkginfo_parcelable.hh
+++ b/src/common/parcel/pkginfo_parcelable.hh
@@ -17,11 +17,6 @@ namespace parcel {
 
 class EXPORT_API PkgInfoParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   PkgInfoParcelable();
   PkgInfoParcelable(uid_t uid, std::vector<package_x *>&& pkg_list, WriteType write_type = None);
   const std::vector<package_x *>& GetPkgInfo();

diff --git a/src/common/parcel/query_parcelable.cc b/src/common/parcel/query_parcelable.cc
index 1d6d12f..ce737fc 100644 (file)
--- a/src/common/parcel/query_parcelable.cc
+++ b/src/common/parcel/query_parcelable.cc
@@ -72,9 +72,5 @@ AbstractDBHandler::OperationType QueryParcelable::GetOpType() {
   return op_type_;
 }
 
-std::unique_ptr<AbstractParcelable> QueryParcelable::Factory::CreateParcel() {
-  return nullptr;
-}
-
 }  // namespace parcel
 }  // namespace pkgmgr_common

diff --git a/src/common/parcel/query_parcelable.hh b/src/common/parcel/query_parcelable.hh
index 63b52c0..2efe307 100644 (file)
--- a/src/common/parcel/query_parcelable.hh
+++ b/src/common/parcel/query_parcelable.hh
@@ -19,11 +19,6 @@ namespace parcel {
 
 class EXPORT_API QueryParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   QueryParcelable();
   QueryParcelable(uid_t uid, std::string query, AbstractDBHandler::DBType db_type, AbstractDBHandler::OperationType op_type);
   QueryParcelable(uid_t uid, std::vector<std::string> query, AbstractDBHandler::DBType db_type, AbstractDBHandler::OperationType op_type);

diff --git a/src/common/parcel/result_parcelable.hh b/src/common/parcel/result_parcelable.hh
index a097163..c42dff3 100644 (file)
--- a/src/common/parcel/result_parcelable.hh
+++ b/src/common/parcel/result_parcelable.hh
@@ -15,11 +15,6 @@ namespace parcel {
 
 class EXPORT_API ResultParcelable : public AbstractParcelable {
  public:
-  class EXPORT_API Factory : IFactory {
-   public:
-    std::unique_ptr<AbstractParcelable> CreateParcel();
-  };
-
   ResultParcelable();
   ResultParcelable(std::vector<std::vector<std::string>>&& results);