projects / platform / core / pim / contacts-service.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
improve line coverage
[platform/core/pim/contacts-service.git] / common / ipc / ctsvc_ipc_group.c
diff --git a/common/ipc/ctsvc_ipc_group.c b/common/ipc/ctsvc_ipc_group.c
index 40c2b9b..a409910 100644 (file)
--- a/common/ipc/ctsvc_ipc_group.c
+++ b/common/ipc/ctsvc_ipc_group.c
@@ -1,7 +1,7 @@
 /*
  * Contacts Service
  *
- * Copyright (c) 2010 - 2012 Samsung Electronics Co., Ltd. All rights reserved.
+ * Copyright (c) 2010 - 2015 Samsung Electronics Co., Ltd. All rights reserved.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,67 +16,86 @@
  * limitations under the License.
  *
  */
-
 #include "ctsvc_internal.h"
 #include "ctsvc_ipc_marshal.h"
 #include "contacts_record.h"
 
-static int __ctsvc_ipc_unmarshal_group(pims_ipc_data_h ipc_data, const char* view_uri, contacts_record_h record);
-static int __ctsvc_ipc_marshal_group(const contacts_record_h record, pims_ipc_data_h ipc_data);
-
-ctsvc_ipc_marshal_record_plugin_cb_s _ctsvc_ipc_record_group_plugin_cb = {
-       .unmarshal_record = __ctsvc_ipc_unmarshal_group,
-       .marshal_record = __ctsvc_ipc_marshal_group
-};
-
 
-static int __ctsvc_ipc_unmarshal_group(pims_ipc_data_h ipc_data, const char* view_uri, contacts_record_h record)
+static int __ctsvc_ipc_unmarshal_group(pims_ipc_data_h ipc_data,
+               const char *view_uri, contacts_record_h record)
 {
-       RETV_IF(ipc_data==NULL,CONTACTS_ERROR_NO_DATA);
-       RETV_IF(record==NULL,CONTACTS_ERROR_NO_DATA);
+       ctsvc_group_s *group_p = (ctsvc_group_s*)record;
 
-       ctsvc_group_s* group_p = (ctsvc_group_s*) record;
+       RETV_IF(NULL == ipc_data, CONTACTS_ERROR_NO_DATA);
+       RETV_IF(NULL == record, CONTACTS_ERROR_NO_DATA);
 
        do {
-               if (ctsvc_ipc_unmarshal_int(ipc_data, &group_p->id) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_int(ipc_data, &group_p->addressbook_id) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_bool(ipc_data, &group_p->is_read_only) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->name) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->extra_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->ringtone_path) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->vibration) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->message_alert) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->image_thumbnail_path) != CONTACTS_ERROR_NONE) break;
+               if (ctsvc_ipc_unmarshal_int(ipc_data, &group_p->id) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_int(ipc_data, &group_p->addressbook_id) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_bool(ipc_data, &group_p->is_read_only) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->name) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->extra_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->ringtone_path) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->vibration) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->message_alert) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_unmarshal_string(ipc_data, &group_p->image_thumbnail_path) != CONTACTS_ERROR_NONE)
+                       break;
 
                return CONTACTS_ERROR_NONE;
 
        } while (0);
-
-       CTS_ERR("_ctsvc_ipc_unmarshal fail");
+       /* LCOV_EXCL_START */
+       ERR("__ctsvc_ipc_unmarshal_group() Fail");
        return CONTACTS_ERROR_INVALID_PARAMETER;
+       /* LCOV_EXCL_STOP */
 }
 
-static int __ctsvc_ipc_marshal_group(const contacts_record_h record, pims_ipc_data_h ipc_data)
+static int __ctsvc_ipc_marshal_group(const contacts_record_h record,
+               pims_ipc_data_h ipc_data)
 {
-       ctsvc_group_s* group_p = (ctsvc_group_s*)record;
-       RETV_IF(ipc_data==NULL,CONTACTS_ERROR_NO_DATA);
-       RETV_IF(group_p==NULL,CONTACTS_ERROR_NO_DATA);
+       ctsvc_group_s *group_p = (ctsvc_group_s*)record;
+
+       RETV_IF(NULL == ipc_data, CONTACTS_ERROR_NO_DATA);
+       RETV_IF(group_p == NULL, CONTACTS_ERROR_NO_DATA);
 
        do {
-               if (ctsvc_ipc_marshal_int((group_p->id),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_int((group_p->addressbook_id),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_bool((group_p->is_read_only),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_string((group_p->name),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_string((group_p->extra_data),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_string((group_p->ringtone_path),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_string((group_p->vibration),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_string((group_p->message_alert),ipc_data) != CONTACTS_ERROR_NONE) break;
-               if (ctsvc_ipc_marshal_string((group_p->image_thumbnail_path),ipc_data) != CONTACTS_ERROR_NONE) break;
+               if (ctsvc_ipc_marshal_int((group_p->id), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_int((group_p->addressbook_id), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_bool((group_p->is_read_only), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_string((group_p->name), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_string((group_p->extra_data), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_string((group_p->ringtone_path), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_string((group_p->vibration), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_string((group_p->message_alert), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
+               if (ctsvc_ipc_marshal_string((group_p->image_thumbnail_path), ipc_data) != CONTACTS_ERROR_NONE)
+                       break;
 
                return CONTACTS_ERROR_NONE;
        } while (0);
-
-       CTS_ERR("_ctsvc_ipc_marshal fail");
+       /* LCOV_EXCL_START */
+       ERR("_ctsvc_ipc_marshal() Fail");
        return CONTACTS_ERROR_INVALID_PARAMETER;
+       /* LCOV_EXCL_STOP */
 }
 
+ctsvc_ipc_marshal_record_plugin_cb_s _ctsvc_ipc_record_group_plugin_cb = {
+       .unmarshal_record = __ctsvc_ipc_unmarshal_group,
+       .marshal_record = __ctsvc_ipc_marshal_group
+};
+
Domain: Service Framework / PIMS;