Fix coding rule for parcelables
authorJunghyun Yeon <jungh.yeon@samsung.com>
Mon, 15 Mar 2021 09:18:45 +0000 (18:18 +0900)
committer연정현/Tizen Platform Lab(SR)/Staff Engineer/삼성전자 <jungh.yeon@samsung.com>
Mon, 15 Mar 2021 22:56:22 +0000 (07:56 +0900)
Signed-off-by: Junghyun Yeon <jungh.yeon@samsung.com>
src/common/parcel/abstract_parcelable.cc
src/common/parcel/appinfo_parcelable.cc
src/common/parcel/certinfo_parcelable.cc
src/common/parcel/command_parcelable.cc
src/common/parcel/filter_parcelable.cc
src/common/parcel/pkginfo_parcelable.cc
src/common/parcel/query_parcelable.cc

index fba5db1..23b7479 100644 (file)
@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "abstract_parcelable.hh"
 
 #include <cstring>
@@ -43,7 +44,8 @@ void AbstractParcelable::WriteInt(tizen_base::Parcel* parcel, int i) const {
   parcel->WriteInt32(i);
 }
 
-void AbstractParcelable::WriteString(tizen_base::Parcel* parcel, const char* str) const {
+void AbstractParcelable::WriteString(tizen_base::Parcel* parcel,
+    const char* str) const {
   if (str == nullptr) {
     parcel->WriteBool(true);
     return;
index 3a6450c..d4c58d4 100644 (file)
@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "appinfo_parcelable.hh"
 
 #include <vector>
@@ -22,7 +23,8 @@
 namespace pkgmgr_common {
 namespace parcel {
 
-void AppInfoParcelable::WriteLabel(tizen_base::Parcel* parcel, GList* label) const {
+void AppInfoParcelable::WriteLabel(tizen_base::Parcel* parcel,
+    GList* label) const {
   int size = g_list_length(label);
   WriteInt(parcel, size);
 
@@ -34,7 +36,8 @@ void AppInfoParcelable::WriteLabel(tizen_base::Parcel* parcel, GList* label) con
   }
 }
 
-void AppInfoParcelable::WriteIcon(tizen_base::Parcel* parcel, GList* icon) const {
+void AppInfoParcelable::WriteIcon(tizen_base::Parcel* parcel,
+    GList* icon) const {
   int size = g_list_length(icon);
   WriteInt(parcel, size);
 
@@ -48,7 +51,8 @@ void AppInfoParcelable::WriteIcon(tizen_base::Parcel* parcel, GList* icon) const
     WriteString(parcel, ptr->dpi);
   }
 }
-void AppInfoParcelable::WriteImage(tizen_base::Parcel* parcel, GList* image) const {
+void AppInfoParcelable::WriteImage(tizen_base::Parcel* parcel,
+    GList* image) const {
   int size = g_list_length(image);
   WriteInt(parcel, size);
 
@@ -59,14 +63,16 @@ void AppInfoParcelable::WriteImage(tizen_base::Parcel* parcel, GList* image) con
     WriteString(parcel, ptr->section);
   }
 }
-void AppInfoParcelable::WriteCategory(tizen_base::Parcel* parcel, GList* category) const {
+void AppInfoParcelable::WriteCategory(tizen_base::Parcel* parcel,
+    GList* category) const {
   int size = g_list_length(category);
   WriteInt(parcel, size);
   for (GList* tmp = category; tmp; tmp = tmp->next) {
     WriteString(parcel, reinterpret_cast<char *>(tmp->data));
   }
 }
-void AppInfoParcelable::WriteMetadata(tizen_base::Parcel* parcel, GList* metadata) const {
+void AppInfoParcelable::WriteMetadata(tizen_base::Parcel* parcel,
+    GList* metadata) const {
   int size = g_list_length(metadata);
   WriteInt(parcel, size);
 
@@ -77,55 +83,20 @@ void AppInfoParcelable::WriteMetadata(tizen_base::Parcel* parcel, GList* metadat
   }
 }
 
-void AppInfoParcelable::WritePermission(tizen_base::Parcel* parcel, GList* permission) const {
-//   int size = g_list_length(permission);
-//    WriteInt(parcel, size);
-//
-//   for (GList* tmp = permission; tmp; tmp = tmp->next) {
-//     label_x* ptr = reinterpret_cast<label_x*>(tmp->data);
-//     ReadString(&label->lang);
-//     ReadString(&label->name);
-//     ReadString(&label->text);
-//   }
-}
+void AppInfoParcelable::WritePermission(tizen_base::Parcel* parcel,
+    GList* permission) const {}
 
-void AppInfoParcelable::WriteLaunchConditions(tizen_base::Parcel* parcel, GList* launchconditions) const {
-//  int size = g_list_length(launchconditions);
-//    WriteInt(parcel, size);
-//
-//  for (GList* tmp = launchconditions; tmp; tmp = tmp->next) {
-//    label_x* ptr = reinterpret_cast<label_x*>(tmp->data);
-//    ReadString(&label->lang);
-//    ReadString(&label->name);
-//    ReadString(&label->text);
-//  }
-}
+void AppInfoParcelable::WriteLaunchConditions(tizen_base::Parcel* parcel,
+    GList* launchconditions) const {}
 
-void AppInfoParcelable::WriteNotification(tizen_base::Parcel* parcel, GList* notification) const {
-//  int size = g_list_length(notification);
-//    WriteInt(parcel, size);
-//
-//  for (GList* tmp = notification; tmp; tmp = tmp->next) {
-//    label_x* ptr = reinterpret_cast<label_x*>(tmp->data);
-//    ReadString(&label->lang);
-//    ReadString(&label->name);
-//    ReadString(&label->text);
-//  }
-}
+void AppInfoParcelable::WriteNotification(tizen_base::Parcel* parcel,
+    GList* notification) const {}
 
-void AppInfoParcelable::WriteDatashare(tizen_base::Parcel* parcel, GList* datashare) const {
-//  int size = g_list_length(datashare);
-//    WriteInt(parcel, size);
-//
-//  for (GList* tmp = datashare; tmp; tmp = tmp->next) {
-//    label_x* ptr = reinterpret_cast<label_x*>(tmp->data);
-//    ReadString(&label->lang);
-//    ReadString(&label->name);
-//    ReadString(&label->text);
-//  }
-}
+void AppInfoParcelable::WriteDatashare(tizen_base::Parcel* parcel,
+    GList* datashare) const {}
 
-void AppInfoParcelable::WriteDatacontrol(tizen_base::Parcel* parcel, GList* datacontrol) const {
+void AppInfoParcelable::WriteDatacontrol(tizen_base::Parcel* parcel,
+    GList* datacontrol) const {
   int size = g_list_length(datacontrol);
   WriteInt(parcel, size);
 
@@ -141,14 +112,16 @@ void AppInfoParcelable::WriteDatacontrol(tizen_base::Parcel* parcel, GList* data
   }
 }
 
-void AppInfoParcelable::WriteBackgroundCategory(tizen_base::Parcel* parcel, GList* backgroundcategory) const {
+void AppInfoParcelable::WriteBackgroundCategory(tizen_base::Parcel* parcel,
+    GList* backgroundcategory) const {
   int size = g_list_length(backgroundcategory);
   WriteInt(parcel, size);
   for (GList* tmp = backgroundcategory; tmp; tmp = tmp->next)
     WriteString(parcel, reinterpret_cast<char *>(tmp->data));
 }
 
-void AppInfoParcelable::WriteAppcontrol(tizen_base::Parcel* parcel, GList* appcontrol) const {
+void AppInfoParcelable::WriteAppcontrol(tizen_base::Parcel* parcel,
+    GList* appcontrol) const {
   int size = g_list_length(appcontrol);
   WriteInt(parcel, size);
 
@@ -165,7 +138,8 @@ void AppInfoParcelable::WriteAppcontrol(tizen_base::Parcel* parcel, GList* appco
   }
 }
 
-void AppInfoParcelable::WriteSplashscreens(tizen_base::Parcel* parcel, GList* splashscreens) const {
+void AppInfoParcelable::WriteSplashscreens(tizen_base::Parcel* parcel,
+    GList* splashscreens) const {
   int size = g_list_length(splashscreens);
   WriteInt(parcel, size);
 
@@ -226,7 +200,8 @@ void AppInfoParcelable::ReadImage(tizen_base::Parcel* parcel, GList** list) {
   }
 }
 
-void AppInfoParcelable::ReadCategory(tizen_base::Parcel* parcel, GList** list) {
+void AppInfoParcelable::ReadCategory(tizen_base::Parcel* parcel,
+    GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -237,12 +212,14 @@ void AppInfoParcelable::ReadCategory(tizen_base::Parcel* parcel, GList** list) {
   }
 }
 
-void AppInfoParcelable::ReadMetadata(tizen_base::Parcel* parcel, GList** list) {
+void AppInfoParcelable::ReadMetadata(tizen_base::Parcel* parcel,
+    GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
   for (int i = 0; i < size; ++i) {
-    metadata_x* metadata = reinterpret_cast<metadata_x*>(calloc(1, sizeof(metadata_x)));
+    metadata_x* metadata =
+        reinterpret_cast<metadata_x*>(calloc(1, sizeof(metadata_x)));
     ReadString(parcel, &metadata->key);
     ReadString(parcel, &metadata->value);
 
@@ -250,68 +227,26 @@ void AppInfoParcelable::ReadMetadata(tizen_base::Parcel* parcel, GList** list) {
   }
 }
 
-void AppInfoParcelable::ReadPermission(tizen_base::Parcel* parcel, GList** list) {
-//  int size = 0;
-//  ReadInt(parcel, &size);
-//
-//   for (int i = 0; i < size; ++i) {
-//     label_x* label = reinterpret_cast<label_x*>(calloc(1, sizeof(label_x)));
-//     ReadString(parcel, &label->lang);
-//     ReadString(parcel, &label->name);
-//     ReadString(parcel, &label->text);
-//
-//     *list = g_list_append(*list, label);
-//   }
-}
+void AppInfoParcelable::ReadPermission(
+    tizen_base::Parcel* parcel, GList** list) {}
 
-void AppInfoParcelable::ReadLaunchConditions(tizen_base::Parcel* parcel, GList** list) {
-//  int size = 0;
-//  ReadInt(parcel, &size);
-//
-//  for (int i = 0; i < size; ++i) {
-//    label_x* label = reinterpret_cast<label_x*>(calloc(1, sizeof(label_x)));
-//    ReadString(parcel, &label->lang);
-//    ReadString(parcel, &label->name);
-//    ReadString(parcel, &label->text);
-//
-//    *list = g_list_append(*list, label);
-//  }
-}
+void AppInfoParcelable::ReadLaunchConditions(
+    tizen_base::Parcel* parcel, GList** list) {}
 
-void AppInfoParcelable::ReadNotification(tizen_base::Parcel* parcel, GList** list) {
-//  int size = 0;
-//  ReadInt(parcel, &size);
-//
-//  for (int i = 0; i < size; ++i) {
-//    label_x* label = reinterpret_cast<label_x*>(calloc(1, sizeof(label_x)));
-//    ReadString(parcel, &label->lang);
-//    ReadString(parcel, &label->name);
-//    ReadString(parcel, &label->text);
-//
-//    *list = g_list_append(*list, label);
-//  }
-}
+void AppInfoParcelable::ReadNotification(
+    tizen_base::Parcel* parcel, GList** list) {}
 
-void AppInfoParcelable::ReadDatashare(tizen_base::Parcel* parcel, GList** list) {
-//  int size = 0;
-//  ReadInt(parcel, &size);
-//
-//  for (int i = 0; i < size; ++i) {
-//    label_x* label = reinterpret_cast<label_x*>(calloc(1, sizeof(label_x)));
-//    ReadString(parcel, &label->lang);
-//    ReadString(parcel, &label->name);
-//    ReadString(parcel, &label->text);
-//
-//    *list = g_list_append(*list, label);
-//  }
-}
+void AppInfoParcelable::ReadDatashare(
+    tizen_base::Parcel* parcel, GList** list) {}
 
-void AppInfoParcelable::ReadDatacontrol(tizen_base::Parcel* parcel, GList** list) {
+void AppInfoParcelable::ReadDatacontrol(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
   for (int i = 0; i < size; ++i) {
-    datacontrol_x* datacontrol = reinterpret_cast<datacontrol_x*>(calloc(1, sizeof(datacontrol_x)));
+    datacontrol_x* datacontrol =
+        reinterpret_cast<datacontrol_x*>(calloc(1, sizeof(datacontrol_x)));
     ReadString(parcel, &datacontrol->providerid);
     ReadString(parcel, &datacontrol->access);
     ReadString(parcel, &datacontrol->type);
@@ -321,13 +256,15 @@ void AppInfoParcelable::ReadDatacontrol(tizen_base::Parcel* parcel, GList** list
     for (int i = 0; i < privilege_size; ++i) {
       char *privilege = nullptr;
       ReadString(parcel, &privilege);
-      datacontrol->privileges = g_list_append(datacontrol->privileges, privilege);
+      datacontrol->privileges =
+          g_list_append(datacontrol->privileges, privilege);
     }
 
     *list = g_list_append(*list, datacontrol);
   }
 }
-void AppInfoParcelable::ReadBackgroundCategory(tizen_base::Parcel* parcel, GList** list) {
+void AppInfoParcelable::ReadBackgroundCategory(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -337,12 +274,14 @@ void AppInfoParcelable::ReadBackgroundCategory(tizen_base::Parcel* parcel, GList
     *list = g_list_append(*list, category);
   }
 }
-void AppInfoParcelable::ReadAppcontrol(tizen_base::Parcel* parcel, GList** list) {
+void AppInfoParcelable::ReadAppcontrol(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
   for (int i = 0; i < size; ++i) {
-    appcontrol_x* appcontrol = reinterpret_cast<appcontrol_x*>(calloc(1, sizeof(appcontrol_x)));
+    appcontrol_x* appcontrol =
+        reinterpret_cast<appcontrol_x*>(calloc(1, sizeof(appcontrol_x)));
     ReadString(parcel, &appcontrol->operation);
     ReadString(parcel, &appcontrol->uri);
     ReadString(parcel, &appcontrol->mime);
@@ -353,18 +292,21 @@ void AppInfoParcelable::ReadAppcontrol(tizen_base::Parcel* parcel, GList** list)
     for (int i = 0; i < privilege_len; ++i) {
       char *privilege = nullptr;
       ReadString(parcel, &privilege);
-      appcontrol->privileges = g_list_append(appcontrol->privileges, privilege);
+      appcontrol->privileges =
+          g_list_append(appcontrol->privileges, privilege);
     }
 
     *list = g_list_append(*list, appcontrol);
   }
 }
-void AppInfoParcelable::ReadSplashscreens(tizen_base::Parcel* parcel, GList** list) {
+void AppInfoParcelable::ReadSplashscreens(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
   for (int i = 0; i < size; ++i) {
-    splashscreen_x* splashscreen = reinterpret_cast<splashscreen_x*>(calloc(1, sizeof(splashscreen_x)));
+    splashscreen_x* splashscreen =
+        reinterpret_cast<splashscreen_x*>(calloc(1, sizeof(splashscreen_x)));
     ReadString(parcel, &splashscreen->src);
     ReadString(parcel, &splashscreen->type);
     ReadString(parcel, &splashscreen->dpi);
@@ -380,8 +322,10 @@ void AppInfoParcelable::ReadSplashscreens(tizen_base::Parcel* parcel, GList** li
 AppInfoParcelable::AppInfoParcelable()
     : AbstractParcelable(0, ParcelableType::AppInfo), auto_release_(true) {}
 
-AppInfoParcelable::AppInfoParcelable(int ret, std::vector<application_x *>&& app_list, bool auto_release)
-    : AbstractParcelable(0, ParcelableType::AppInfo, ret), app_list_(std::move(app_list)), auto_release_(auto_release) {}
+AppInfoParcelable::AppInfoParcelable(int ret,
+    std::vector<application_x *>&& app_list, bool auto_release)
+    : AbstractParcelable(0, ParcelableType::AppInfo, ret),
+    app_list_(std::move(app_list)), auto_release_(auto_release) {}
 
 AppInfoParcelable::~AppInfoParcelable() {
   if (!auto_release_)
@@ -459,10 +403,6 @@ application_x *AppInfoParcelable::ReadApplication(tizen_base::Parcel* parcel) {
   ReadImage(parcel, &application->image);
   ReadCategory(parcel, &application->category);
   ReadMetadata(parcel, &application->metadata);
-  //ReadPermission(parcel, &application->permission);
-  //ReadLaunchConditions(parcel, &application->launchconditions);
-  //ReadNotification(parcel, &application->notification);
-  //ReadDatashare(parcel, &application->datashare);
   ReadDatacontrol(parcel, &application->datacontrol);
   ReadBackgroundCategory(parcel, &application->background_category);
   ReadAppcontrol(parcel, &application->appcontrol);
@@ -471,7 +411,8 @@ application_x *AppInfoParcelable::ReadApplication(tizen_base::Parcel* parcel) {
   return application;
 }
 
-bool AppInfoParcelable::WriteApplication(tizen_base::Parcel* parcel, application_x *application) const {
+bool AppInfoParcelable::WriteApplication(tizen_base::Parcel* parcel,
+    application_x *application) const {
   WriteString(parcel, application->appid);
   WriteString(parcel, application->exec);
   WriteString(parcel, application->nodisplay);
@@ -528,10 +469,6 @@ bool AppInfoParcelable::WriteApplication(tizen_base::Parcel* parcel, application
   WriteImage(parcel, application->image);
   WriteCategory(parcel, application->category);
   WriteMetadata(parcel, application->metadata);
-  //WritePermission(parcel, application->permission);
-  //WriteLaunchConditions(parcel, application->launchconditions);
-  //WriteNotification(parcel, application->notification);
-  //WriteDatashare(parcel, application->datashare);
   WriteDatacontrol(parcel, application->datacontrol);
   WriteBackgroundCategory(parcel, application->background_category);
   WriteAppcontrol(parcel, application->appcontrol);
@@ -544,9 +481,8 @@ void AppInfoParcelable::WriteToParcel(tizen_base::Parcel* parcel) const {
   AbstractParcelable::WriteToParcel(parcel);
   WriteInt(parcel, app_list_.size());
 
-  for (const auto app : app_list_) {
+  for (const auto app : app_list_)
     WriteApplication(parcel, app);
-  }
 }
 
 void AppInfoParcelable::ReadFromParcel(tizen_base::Parcel* parcel) {
@@ -555,9 +491,8 @@ void AppInfoParcelable::ReadFromParcel(tizen_base::Parcel* parcel) {
 
   ReadInt(parcel, &len);
 
-  for (int i = 0; i < len ; ++i) {
+  for (int i = 0; i < len ; ++i)
     app_list_.emplace_back(ReadApplication(parcel));
-  }
 }
 
 }  // namespace parcel
index a349fb9..1269c0e 100644 (file)
@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "certinfo_parcelable.hh"
 
 #include <vector>
@@ -29,13 +30,15 @@ CertInfoParcelable::CertInfoParcelable()
       pkgid_(""),
       auto_release_(true) {}
 
-CertInfoParcelable::CertInfoParcelable(uid_t uid, pkgmgr_certinfo_x* cert_info, bool auto_release)
+CertInfoParcelable::CertInfoParcelable(
+    uid_t uid, pkgmgr_certinfo_x* cert_info, bool auto_release)
     : AbstractParcelable(uid, ParcelableType::CertInfo),
       cert_info_(cert_info),
       pkgid_(""),
       auto_release_(auto_release) {}
 
-CertInfoParcelable::CertInfoParcelable(int ret, pkgmgr_certinfo_x* cert_info, bool auto_release)
+CertInfoParcelable::CertInfoParcelable(
+    int ret, pkgmgr_certinfo_x* cert_info, bool auto_release)
     : AbstractParcelable(0, ParcelableType::CertInfo, ret),
       cert_info_(cert_info),
       pkgid_(""),
index 87a4b3e..83649b8 100644 (file)
@@ -15,6 +15,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "command_parcelable.hh"
 
 #include <vector>
@@ -22,7 +23,8 @@
 namespace pkgmgr_common {
 namespace parcel {
 CommandParcelable::CommandParcelable()
-    : AbstractParcelable(0, ParcelableType::Command), uid_(0), cmd_(CommandType::Unknown) {}
+    : AbstractParcelable(0, ParcelableType::Command),
+    uid_(0), cmd_(CommandType::Unknown) {}
 
 CommandParcelable::CommandParcelable(uid_t uid, CommandType cmd)
     : AbstractParcelable(0, ParcelableType::Command), uid_(uid), cmd_(cmd) {}
index a0de5e8..c73896f 100644 (file)
@@ -42,7 +42,6 @@ FilterParcelable::~FilterParcelable() {
 
   if (filter_ != nullptr)
     pkgmgrinfo_pkginfo_filter_destroy(filter_);
-
 }
 
 const pkgmgrinfo_filter_x *FilterParcelable::GetFilter() { return filter_; }
index 5fa5d6f..fb9969e 100644 (file)
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "pkginfo_parcelable.hh"
 
 #include <vector>
 
-#include "pkgmgrinfo_private.h"
-
 #include "appinfo_parcelable.hh"
 
+#include "pkgmgrinfo_private.h"
+
 namespace pkgmgr_common {
 namespace parcel {
 
-void PkgInfoParcelable::WriteIcon(tizen_base::Parcel* parcel, GList* icon) const {
+void PkgInfoParcelable::WriteIcon(tizen_base::Parcel* parcel,
+    GList* icon) const {
   int size = g_list_length(icon);
   WriteInt(parcel, size);
 
@@ -39,7 +41,8 @@ void PkgInfoParcelable::WriteIcon(tizen_base::Parcel* parcel, GList* icon) const
   }
 }
 
-void PkgInfoParcelable::WriteLabel(tizen_base::Parcel* parcel, GList* label) const {
+void PkgInfoParcelable::WriteLabel(tizen_base::Parcel* parcel,
+    GList* label) const {
   int size = g_list_length(label);
   WriteInt(parcel, size);
 
@@ -51,7 +54,8 @@ void PkgInfoParcelable::WriteLabel(tizen_base::Parcel* parcel, GList* label) con
   }
 }
 
-void PkgInfoParcelable::WriteAuthor(tizen_base::Parcel* parcel, GList* author) const {
+void PkgInfoParcelable::WriteAuthor(tizen_base::Parcel* parcel,
+    GList* author) const {
   int size = g_list_length(author);
   WriteInt(parcel, size);
 
@@ -64,7 +68,8 @@ void PkgInfoParcelable::WriteAuthor(tizen_base::Parcel* parcel, GList* author) c
   }
 }
 
-void PkgInfoParcelable::WriteDescription(tizen_base::Parcel* parcel, GList* description) const {
+void PkgInfoParcelable::WriteDescription(tizen_base::Parcel* parcel,
+    GList* description) const {
   int size = g_list_length(description);
   WriteInt(parcel, size);
 
@@ -76,11 +81,13 @@ void PkgInfoParcelable::WriteDescription(tizen_base::Parcel* parcel, GList* desc
   }
 }
 
-void PkgInfoParcelable::WriteLicense(tizen_base::Parcel* parcel, GList* license) const {
+void PkgInfoParcelable::WriteLicense(tizen_base::Parcel* parcel,
+    GList* license) const {
   return;
 }
 
-void PkgInfoParcelable::WritePrivileges(tizen_base::Parcel* parcel, GList* privileges) const {
+void PkgInfoParcelable::WritePrivileges(tizen_base::Parcel* parcel,
+    GList* privileges) const {
   int size = g_list_length(privileges);
   WriteInt(parcel, size);
 
@@ -105,8 +112,8 @@ void PkgInfoParcelable::WriteAppdefinedPrivileges(tizen_base::Parcel* parcel,
   }
 }
 
-void PkgInfoParcelable::WriteProvidesAppdefinedPrivileges(tizen_base::Parcel* parcel,
-    GList* provides_appdefined_privileges) const {
+void PkgInfoParcelable::WriteProvidesAppdefinedPrivileges(
+    tizen_base::Parcel* parcel, GList* provides_appdefined_privileges) const {
   int size = g_list_length(provides_appdefined_privileges);
   WriteInt(parcel, size);
 
@@ -119,7 +126,8 @@ void PkgInfoParcelable::WriteProvidesAppdefinedPrivileges(tizen_base::Parcel* pa
   }
 }
 
-void PkgInfoParcelable::WriteApplication(tizen_base::Parcel* parcel, GList* application) const {
+void PkgInfoParcelable::WriteApplication(tizen_base::Parcel* parcel,
+    GList* application) const {
   std::vector<application_x *> app_vt;
 
   for (GList* tmp = application; tmp; tmp = tmp->next) {
@@ -146,7 +154,8 @@ void PkgInfoParcelable::WriteDeviceprofile(tizen_base::Parcel* parcel,
     WriteString(parcel, reinterpret_cast<char *>(tmp->data));
 }
 
-void PkgInfoParcelable::WriteDependencies(tizen_base::Parcel* parcel, GList* dependencies) const {
+void PkgInfoParcelable::WriteDependencies(tizen_base::Parcel* parcel,
+    GList* dependencies) const {
   int size = g_list_length(dependencies);
   WriteInt(parcel, size);
 
@@ -158,7 +167,8 @@ void PkgInfoParcelable::WriteDependencies(tizen_base::Parcel* parcel, GList* dep
   }
 }
 
-void PkgInfoParcelable::WritePlugin(tizen_base::Parcel* parcel, GList* plugin) const {
+void PkgInfoParcelable::WritePlugin(tizen_base::Parcel* parcel,
+    GList* plugin) const {
   int size = g_list_length(plugin);
   WriteInt(parcel, size);
 
@@ -171,7 +181,8 @@ void PkgInfoParcelable::WritePlugin(tizen_base::Parcel* parcel, GList* plugin) c
   }
 }
 
-void PkgInfoParcelable::WritePackage(tizen_base::Parcel* parcel, package_x *package) const {
+void PkgInfoParcelable::WritePackage(tizen_base::Parcel* parcel,
+    package_x *package) const {
   WriteString(parcel, package->for_all_users);
   WriteString(parcel, package->package);
   WriteString(parcel, package->version);
@@ -206,12 +217,11 @@ void PkgInfoParcelable::WritePackage(tizen_base::Parcel* parcel, package_x *pack
   WriteLabel(parcel, package->label);
   WriteAuthor(parcel, package->author);
   WriteDescription(parcel, package->description);
-  //WriteLicense(parcel, package->license);
   WritePrivileges(parcel, package->privileges);
   WriteAppdefinedPrivileges(parcel, package->appdefined_privileges);
-  WriteProvidesAppdefinedPrivileges(parcel, package->provides_appdefined_privileges);
+  WriteProvidesAppdefinedPrivileges(parcel,
+      package->provides_appdefined_privileges);
   WriteApplication(parcel, package->application);
-  //WriteCompatibility(parcel, package->compatibility);
   WriteDeviceprofile(parcel, package->deviceprofile);
   WriteDependencies(parcel, package->dependencies);
   WritePlugin(parcel, package->plugin);
@@ -253,7 +263,8 @@ void PkgInfoParcelable::ReadAuthor(tizen_base::Parcel* parcel, GList** list) {
   ReadInt(parcel, &size);
 
   for (int i = 0; i < size; ++i) {
-    author_x* author = reinterpret_cast<author_x*>(calloc(1, sizeof(author_x)));
+    author_x* author = reinterpret_cast<author_x*>(
+        calloc(1, sizeof(author_x)));
     ReadString(parcel, &author->email);
     ReadString(parcel, &author->href);
     ReadString(parcel, &author->text);
@@ -263,7 +274,8 @@ void PkgInfoParcelable::ReadAuthor(tizen_base::Parcel* parcel, GList** list) {
   }
 }
 
-void PkgInfoParcelable::ReadDescription(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadDescription(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -282,7 +294,8 @@ void PkgInfoParcelable::ReadLicense(tizen_base::Parcel* parcel, GList** list) {
   return;
 }
 
-void PkgInfoParcelable::ReadPrivileges(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadPrivileges(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -296,7 +309,8 @@ void PkgInfoParcelable::ReadPrivileges(tizen_base::Parcel* parcel, GList** list)
   }
 }
 
-void PkgInfoParcelable::ReadAppdefinedPrivileges(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadAppdefinedPrivileges(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -312,8 +326,8 @@ void PkgInfoParcelable::ReadAppdefinedPrivileges(tizen_base::Parcel* parcel, GLi
   }
 }
 
-void PkgInfoParcelable::ReadProvidesAppdefinedPrivileges(tizen_base::Parcel* parcel,
-    GList** list) {
+void PkgInfoParcelable::ReadProvidesAppdefinedPrivileges(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -329,7 +343,8 @@ void PkgInfoParcelable::ReadProvidesAppdefinedPrivileges(tizen_base::Parcel* par
   }
 }
 
-void PkgInfoParcelable::ReadApplication(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadApplication(
+    tizen_base::Parcel* parcel, GList** list) {
   AppInfoParcelable apps;
   parcel->ReadParcelable(&apps);
 
@@ -339,11 +354,13 @@ void PkgInfoParcelable::ReadApplication(tizen_base::Parcel* parcel, GList** list
     *list = g_list_append(*list, application);
 }
 
-void PkgInfoParcelable::ReadCompatibility(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadCompatibility(
+    tizen_base::Parcel* parcel, GList** list) {
   return;
 }
 
-void PkgInfoParcelable::ReadDeviceprofile(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadDeviceprofile(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -354,7 +371,8 @@ void PkgInfoParcelable::ReadDeviceprofile(tizen_base::Parcel* parcel, GList** li
   }
 }
 
-void PkgInfoParcelable::ReadDependencies(tizen_base::Parcel* parcel, GList** list) {
+void PkgInfoParcelable::ReadDependencies(
+    tizen_base::Parcel* parcel, GList** list) {
   int size = 0;
   ReadInt(parcel, &size);
 
@@ -374,7 +392,8 @@ void PkgInfoParcelable::ReadPlugin(tizen_base::Parcel* parcel, GList** list) {
   ReadInt(parcel, &size);
 
   for (int i = 0; i < size; ++i) {
-    plugin_x* plugin = reinterpret_cast<plugin_x*>(calloc(1, sizeof(plugin_x)));
+    plugin_x* plugin = reinterpret_cast<plugin_x*>(
+        calloc(1, sizeof(plugin_x)));
     ReadString(parcel, &plugin->pkgid);
     ReadString(parcel, &plugin->appid);
     ReadString(parcel, &plugin->plugin_type);
@@ -422,12 +441,11 @@ package_x *PkgInfoParcelable::ReadPackage(tizen_base::Parcel* parcel) {
   ReadLabel(parcel, &package->label);
   ReadAuthor(parcel, &package->author);
   ReadDescription(parcel, &package->description);
-  //ReadLicense(parcel, &package->license);
   ReadPrivileges(parcel, &package->privileges);
   ReadAppdefinedPrivileges(parcel, &package->appdefined_privileges);
-  ReadProvidesAppdefinedPrivileges(parcel, &package->provides_appdefined_privileges);
+  ReadProvidesAppdefinedPrivileges(parcel,
+      &package->provides_appdefined_privileges);
   ReadApplication(parcel, &package->application);
-  //ReadCompatibility(parcel, &package->compatibility);
   ReadDeviceprofile(parcel, &package->deviceprofile);
   ReadDependencies(parcel, &package->dependencies);
   ReadPlugin(parcel, &package->plugin);
@@ -436,13 +454,21 @@ package_x *PkgInfoParcelable::ReadPackage(tizen_base::Parcel* parcel) {
 }
 
 PkgInfoParcelable::PkgInfoParcelable()
-    : AbstractParcelable(0, ParcelableType::PkgInfo), write_type_(WriteType::None), auto_release_(true) {}
-
-PkgInfoParcelable::PkgInfoParcelable(uid_t uid, std::vector<package_x *>&& pkg_list, WriteType write_type, bool auto_release)
-    : AbstractParcelable(uid, ParcelableType::PkgInfo), pkg_list_(std::move(pkg_list)), write_type_(write_type), auto_release_(auto_release) {}
-
-PkgInfoParcelable::PkgInfoParcelable(int ret, std::vector<package_x *>&& pkg_list, bool auto_release)
-    : AbstractParcelable(0, ParcelableType::PkgInfo, ret), pkg_list_(std::move(pkg_list)), write_type_(WriteType::None), auto_release_(auto_release) {}
+    : AbstractParcelable(0, ParcelableType::PkgInfo),
+    write_type_(WriteType::None), auto_release_(true) {}
+
+PkgInfoParcelable::PkgInfoParcelable(
+    uid_t uid, std::vector<package_x *>&& pkg_list,
+    WriteType write_type, bool auto_release)
+    : AbstractParcelable(uid, ParcelableType::PkgInfo),
+    pkg_list_(std::move(pkg_list)), write_type_(write_type),
+    auto_release_(auto_release) {}
+
+PkgInfoParcelable::PkgInfoParcelable(
+    int ret, std::vector<package_x *>&& pkg_list, bool auto_release)
+    : AbstractParcelable(0, ParcelableType::PkgInfo, ret),
+    pkg_list_(std::move(pkg_list)), write_type_(WriteType::None),
+    auto_release_(auto_release) {}
 
 PkgInfoParcelable::~PkgInfoParcelable() {
   if (!auto_release_)
index ce737fc..961a9c1 100644 (file)
@@ -15,6 +15,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 #include "query_parcelable.hh"
 
 #include <vector>
@@ -25,15 +26,23 @@ namespace pkgmgr_common {
 namespace parcel {
 
 QueryParcelable::QueryParcelable()
-    : AbstractParcelable(0, ParcelableType::Query), db_type_(AbstractDBHandler::DBType::DB_TYPE_NONE), op_type_(AbstractDBHandler::OperationType::OPERATION_TYPE_NONE) {}
+    : AbstractParcelable(0, ParcelableType::Query),
+    db_type_(AbstractDBHandler::DBType::DB_TYPE_NONE),
+    op_type_(AbstractDBHandler::OperationType::OPERATION_TYPE_NONE) {}
 
 
 QueryParcelable::QueryParcelable(uid_t uid, std::string query,
-    AbstractDBHandler::DBType db_type, AbstractDBHandler::OperationType op_type)
-    : AbstractParcelable(0, ParcelableType::Query), queries_(std::vector<std::string>{query}), db_type_(db_type), op_type_(op_type) {}
+    AbstractDBHandler::DBType db_type,
+    AbstractDBHandler::OperationType op_type)
+    : AbstractParcelable(0, ParcelableType::Query),
+    queries_(std::vector<std::string>{query}),
+    db_type_(db_type), op_type_(op_type) {}
 
-QueryParcelable::QueryParcelable(uid_t uid, std::vector<std::string> queries, AbstractDBHandler::DBType db_type, AbstractDBHandler::OperationType op_type)
-    : AbstractParcelable(uid, ParcelableType::Query), queries_(queries), db_type_(db_type), op_type_(op_type) {}
+QueryParcelable::QueryParcelable(uid_t uid, std::vector<std::string> queries,
+    AbstractDBHandler::DBType db_type,
+    AbstractDBHandler::OperationType op_type)
+    : AbstractParcelable(uid, ParcelableType::Query),
+    queries_(queries), db_type_(db_type), op_type_(op_type) {}
 
 void QueryParcelable::WriteToParcel(tizen_base::Parcel* parcel) const {
   AbstractParcelable::WriteToParcel(parcel);