From e1b326dfa40a772e11f2b37e58352b1ac2c2b7a8 Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Tue, 23 Mar 2021 09:58:03 +0900 Subject: [PATCH 01/16] Fix coding indentation Change-Id: Ide4fbf38bbb247851148f44d8914df047bc59c4c Signed-off-by: Jihoon Kim --- client/src/sticker_data.c | 2 +- client/src/sticker_provider.c | 4 +- receiver/src/ft.cpp | 2 +- server/stickerd_data_manager.c | 28 ++--- sticker-parser/sticker-parser.c | 4 +- tests/mock/fff.h | 240 ++++++++++++++++++++-------------------- 6 files changed, 140 insertions(+), 140 deletions(-) diff --git a/client/src/sticker_data.c b/client/src/sticker_data.c index 2b47114..8ed40e6 100644 --- a/client/src/sticker_data.c +++ b/client/src/sticker_data.c @@ -67,7 +67,7 @@ static char* _make_absolute_path(const char *relative_path) goto cleanup; } - if(relative_path[0] == '/') + if (relative_path[0] == '/') snprintf(file_path, path_len, "%s%s",app_path, relative_path); else snprintf(file_path, path_len, "%s%s%s",app_path, "/", relative_path); //LCOV_EXCL_LINE diff --git a/client/src/sticker_provider.c b/client/src/sticker_provider.c index 079eaaa..b08b024 100644 --- a/client/src/sticker_provider.c +++ b/client/src/sticker_provider.c @@ -180,7 +180,7 @@ EXPORT_API int sticker_provider_insert_data_by_json_file(sticker_provider_h prov goto cleanup; } - if(json_path[0] == '/') + if (json_path[0] == '/') snprintf(file_path, path_len, "%s%s",app_path, json_path); else snprintf(file_path, path_len, "%s%s%s",app_path, "/", json_path); @@ -437,7 +437,7 @@ EXPORT_API int sticker_provider_set_group_image(sticker_provider_h provider_hand goto cleanup; } - if(uri[0] == '/') + if (uri[0] == '/') snprintf(file_path, path_len, "%s%s",app_path, uri); else snprintf(file_path, path_len, "%s%s%s",app_path, "/", uri); //LCOV_EXCL_LINE diff --git a/receiver/src/ft.cpp b/receiver/src/ft.cpp index 9a717ee..12d3922 100644 --- a/receiver/src/ft.cpp +++ b/receiver/src/ft.cpp @@ -794,7 +794,7 @@ on_data_received(sap_socket_h socket, unsigned short int channel_id, unsigned in int result_code = json_object_get_int_member(root_obj, "resultCode"); STLOGD("result : %s, resultCode : %d", result.c_str(), result_code); - if(result_code == SYNC_START_RSP_SUCCESS) { + if (result_code == SYNC_START_RSP_SUCCESS) { send_sync_start_response(result_code); if (current_request.category == string("bitmoji")) { STLOGD("Delete all bitmoji stickers"); diff --git a/server/stickerd_data_manager.c b/server/stickerd_data_manager.c index 1e8030c..3210959 100644 --- a/server/stickerd_data_manager.c +++ b/server/stickerd_data_manager.c @@ -436,7 +436,7 @@ static int _check_file_exist(const char *app_id, const char *path) goto cleanup; } - if(path[0] == '/') + if (path[0] == '/') snprintf(file_path, path_len, "%s%s",app_path, path); else snprintf(file_path, path_len, "%s%s%s",app_path, "/", path); @@ -525,9 +525,9 @@ static int _file_copy(const char *src, const char *dest) goto cleanup; } - while((size = read(fd, buf, 4096))) { + while ((size = read(fd, buf, 4096))) { if (size == -1) { - if(errno == EINTR) + if (errno == EINTR) continue; tmp_err = errno; @@ -535,8 +535,8 @@ static int _file_copy(const char *src, const char *dest) goto cleanup; } - while(write(n_fd, buf, size) == -1) { - if(errno == EINTR) { + while (write(n_fd, buf, size) == -1) { + if (errno == EINTR) { continue; } else { tmp_err = errno; @@ -995,7 +995,7 @@ cleanup: if (ret != STICKERD_SERVER_ERROR_NONE) LOGE("Failed to send insert result to client"); - if(body) + if (body) g_variant_unref(body); return ret; @@ -1396,7 +1396,7 @@ int stickerd_get_all_sticker_info(GVariant *parameters, GVariant **reply_body) ret = stickerd_db_get_record_id(STICKER_DB_STICKER_ALL, &id_list, NULL, app_id, offset, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get all sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1432,7 +1432,7 @@ int stickerd_get_sticker_info_by_app_id(GVariant *parameters, GVariant **reply_b ret = stickerd_db_get_record_id(STICKER_DB_STICKER_APPID, &id_list, NULL, app_id, offset, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get all sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1468,7 +1468,7 @@ int stickerd_get_sticker_info_by_type(GVariant *parameters, GVariant **reply_bod ret = stickerd_db_get_record_id(STICKER_DB_STICKER_TYPE, &id_list, &type, app_id, offset, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get all sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1505,7 +1505,7 @@ int stickerd_get_sticker_info_by_group(GVariant *parameters, GVariant **reply_bo ret = stickerd_db_get_record_id(STICKER_DB_STICKER_GROUP, &id_list, (void *)group, app_id, offset, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get all sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1542,7 +1542,7 @@ int stickerd_get_sticker_info_by_keyword(GVariant *parameters, GVariant **reply_ ret = stickerd_db_get_record_id(STICKER_DB_STICKER_KEYWORD, &id_list, (void *)keyword, app_id, offset, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get all sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1578,7 +1578,7 @@ int stickerd_get_sticker_info_by_display_type(GVariant *parameters, GVariant **r ret = stickerd_db_get_record_id(STICKER_DB_STICKER_DISP_TYPE, &id_list, &type, app_id, offset, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get all sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1709,7 +1709,7 @@ int stickerd_get_recent_sticker_info(GVariant *parameters, GVariant **reply_body ret = stickerd_db_get_record_id(STICKER_DB_STICKER_RECENT_HISTORY, &id_list, NULL, NULL, 0, count); if (ret != STICKERD_SERVER_ERROR_NONE) { LOGE("Failed to get recent sticker id"); - if(id_list) + if (id_list) g_list_free_full(id_list, free); return STICKERD_SERVER_ERROR_OPERATION_FAILED; } @@ -1854,7 +1854,7 @@ int stickerd_set_group_image(GVariant *parameters, GVariant **reply_body) if (type == STICKER_URI_TYPE_LOCAL_PATH) { if (_check_file_exist(app_id, uri) == 0) { uri = _convert_sticker_uri(uri, app_id); - if(!uri) { + if (!uri) { LOGE("failed to copy sticker file"); return STICKERD_SERVER_ERROR_FILE_EXISTS; } diff --git a/sticker-parser/sticker-parser.c b/sticker-parser/sticker-parser.c index 2eb877a..321e141 100644 --- a/sticker-parser/sticker-parser.c +++ b/sticker-parser/sticker-parser.c @@ -346,7 +346,7 @@ static int __file_copy(const char *src, const char *dest) while((size = read(fd, buf, 4096))) { if (size == -1) { - if(errno == EINTR) + if (errno == EINTR) continue; tmp_err = errno; @@ -355,7 +355,7 @@ static int __file_copy(const char *src, const char *dest) } while(write(n_fd, buf, size) == -1) { - if(errno == EINTR) { + if (errno == EINTR) { continue; } else { tmp_err = errno; diff --git a/tests/mock/fff.h b/tests/mock/fff.h index f8d6d5f..93e34e5 100644 --- a/tests/mock/fff.h +++ b/tests/mock/fff.h @@ -93,7 +93,7 @@ SOFTWARE. #define RETURN_FAKE_RESULT(FUNCNAME) \ if (FUNCNAME##_fake.return_val_seq_len){ /* then its a sequence */ \ - if(FUNCNAME##_fake.return_val_seq_idx < FUNCNAME##_fake.return_val_seq_len) { \ + if (FUNCNAME##_fake.return_val_seq_idx < FUNCNAME##_fake.return_val_seq_len) { \ SAVE_RET_HISTORY(FUNCNAME, FUNCNAME##_fake.return_val_seq[FUNCNAME##_fake.return_val_seq_idx]) \ return FUNCNAME##_fake.return_val_seq[FUNCNAME##_fake.return_val_seq_idx++]; \ } \ @@ -138,7 +138,7 @@ FFF_END_EXTERN_C memset(fff.call_history, 0, sizeof(fff.call_history)); #define REGISTER_CALL(function) \ - if(fff.call_history_idx < FFF_CALL_HISTORY_LEN) \ + if (fff.call_history_idx < FFF_CALL_HISTORY_LEN) \ fff.call_history[fff.call_history_idx++] = (fff_function_t)function; #define DECLARE_FAKE_VOID_FUNC0(FUNCNAME) \ @@ -155,7 +155,7 @@ FFF_END_EXTERN_C #define DEFINE_FAKE_VOID_FUNC0(FUNCNAME) \ FUNCNAME##_Fake FUNCNAME##_fake; \ void FUNCNAME(void){ \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ } \ else{ \ HISTORY_DROPPED(FUNCNAME); \ @@ -195,7 +195,7 @@ FFF_END_EXTERN_C FUNCNAME##_Fake FUNCNAME##_fake; \ void FUNCNAME(ARG0_TYPE arg0){ \ SAVE_ARG(FUNCNAME, 0); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ } \ else{ \ @@ -238,7 +238,7 @@ FFF_END_EXTERN_C void FUNCNAME(ARG0_TYPE arg0, ARG1_TYPE arg1){ \ SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ } \ @@ -284,7 +284,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -333,7 +333,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -385,7 +385,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -440,7 +440,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -498,7 +498,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -559,7 +559,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -623,7 +623,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -690,7 +690,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -760,7 +760,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -833,7 +833,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -909,7 +909,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -988,7 +988,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1070,7 +1070,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1155,7 +1155,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1243,7 +1243,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1334,7 +1334,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1428,7 +1428,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ SAVE_ARG(FUNCNAME, 18); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1525,7 +1525,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 17); \ SAVE_ARG(FUNCNAME, 18); \ SAVE_ARG(FUNCNAME, 19); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1585,7 +1585,7 @@ FFF_END_EXTERN_C #define DEFINE_FAKE_VALUE_FUNC0(RETURN_TYPE, FUNCNAME) \ FUNCNAME##_Fake FUNCNAME##_fake; \ RETURN_TYPE FUNCNAME(void){ \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ } \ else{ \ HISTORY_DROPPED(FUNCNAME); \ @@ -1633,7 +1633,7 @@ FFF_END_EXTERN_C FUNCNAME##_Fake FUNCNAME##_fake; \ RETURN_TYPE FUNCNAME(ARG0_TYPE arg0){ \ SAVE_ARG(FUNCNAME, 0); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ } \ else{ \ @@ -1684,7 +1684,7 @@ FFF_END_EXTERN_C RETURN_TYPE FUNCNAME(ARG0_TYPE arg0, ARG1_TYPE arg1){ \ SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ } \ @@ -1738,7 +1738,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1795,7 +1795,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1855,7 +1855,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1918,7 +1918,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -1984,7 +1984,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2053,7 +2053,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2125,7 +2125,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2200,7 +2200,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2278,7 +2278,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2359,7 +2359,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2443,7 +2443,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2530,7 +2530,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2620,7 +2620,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2713,7 +2713,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2809,7 +2809,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -2908,7 +2908,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3010,7 +3010,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ SAVE_ARG(FUNCNAME, 18); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3115,7 +3115,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 17); \ SAVE_ARG(FUNCNAME, 18); \ SAVE_ARG(FUNCNAME, 19); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3181,7 +3181,7 @@ FFF_END_EXTERN_C FUNCNAME##_Fake FUNCNAME##_fake; \ void FUNCNAME(ARG0_TYPE arg0, ...){ \ SAVE_ARG(FUNCNAME, 0); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ } \ else{ \ @@ -3203,7 +3203,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg0); \ FUNCNAME##_fake.custom_fake(arg0, ap); \ @@ -3235,7 +3235,7 @@ FFF_END_EXTERN_C void FUNCNAME(ARG0_TYPE arg0, ARG1_TYPE arg1, ...){ \ SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ } \ @@ -3258,7 +3258,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg1); \ FUNCNAME##_fake.custom_fake(arg0, arg1, ap); \ @@ -3292,7 +3292,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3316,7 +3316,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg2); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, ap); \ @@ -3352,7 +3352,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3377,7 +3377,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg3); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, ap); \ @@ -3415,7 +3415,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3441,7 +3441,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg4); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, ap); \ @@ -3481,7 +3481,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3508,7 +3508,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg5); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, ap); \ @@ -3550,7 +3550,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3578,7 +3578,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg6); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, ap); \ @@ -3622,7 +3622,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3651,7 +3651,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg7); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, ap); \ @@ -3697,7 +3697,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3727,7 +3727,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg8); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, ap); \ @@ -3775,7 +3775,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3806,7 +3806,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg9); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, ap); \ @@ -3856,7 +3856,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3888,7 +3888,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg10); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, ap); \ @@ -3940,7 +3940,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -3973,7 +3973,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg11); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, ap); \ @@ -4027,7 +4027,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4061,7 +4061,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg12); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, ap); \ @@ -4117,7 +4117,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4152,7 +4152,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg13); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, ap); \ @@ -4210,7 +4210,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4246,7 +4246,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg14); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, ap); \ @@ -4306,7 +4306,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4343,7 +4343,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg15); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, ap); \ @@ -4405,7 +4405,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4443,7 +4443,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg16); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, arg16, ap); \ @@ -4507,7 +4507,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4546,7 +4546,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg17); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, arg16, arg17, ap); \ @@ -4612,7 +4612,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ SAVE_ARG(FUNCNAME, 18); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4652,7 +4652,7 @@ FFF_END_EXTERN_C va_end(ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ va_list ap; \ va_start(ap, arg18); \ FUNCNAME##_fake.custom_fake(arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, arg16, arg17, arg18, ap); \ @@ -4684,7 +4684,7 @@ FFF_END_EXTERN_C FUNCNAME##_Fake FUNCNAME##_fake; \ RETURN_TYPE FUNCNAME(ARG0_TYPE arg0, ...){ \ SAVE_ARG(FUNCNAME, 0); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ } \ else{ \ @@ -4711,7 +4711,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg0); \ @@ -4749,7 +4749,7 @@ FFF_END_EXTERN_C RETURN_TYPE FUNCNAME(ARG0_TYPE arg0, ARG1_TYPE arg1, ...){ \ SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ } \ @@ -4777,7 +4777,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg1); \ @@ -4817,7 +4817,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 0); \ SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4846,7 +4846,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg2); \ @@ -4888,7 +4888,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 1); \ SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4918,7 +4918,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg3); \ @@ -4962,7 +4962,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 2); \ SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -4993,7 +4993,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg4); \ @@ -5039,7 +5039,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 3); \ SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5071,7 +5071,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg5); \ @@ -5119,7 +5119,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 4); \ SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5152,7 +5152,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg6); \ @@ -5202,7 +5202,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 5); \ SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5236,7 +5236,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg7); \ @@ -5288,7 +5288,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 6); \ SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5323,7 +5323,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg8); \ @@ -5377,7 +5377,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 7); \ SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5413,7 +5413,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg9); \ @@ -5469,7 +5469,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 8); \ SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5506,7 +5506,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg10); \ @@ -5564,7 +5564,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 9); \ SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5602,7 +5602,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg11); \ @@ -5662,7 +5662,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 10); \ SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5701,7 +5701,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg12); \ @@ -5763,7 +5763,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 11); \ SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5803,7 +5803,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg13); \ @@ -5867,7 +5867,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 12); \ SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -5908,7 +5908,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg14); \ @@ -5974,7 +5974,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 13); \ SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -6016,7 +6016,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg15); \ @@ -6084,7 +6084,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 14); \ SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -6127,7 +6127,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, arg16, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg16); \ @@ -6197,7 +6197,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 15); \ SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -6241,7 +6241,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, arg16, arg17, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg17); \ @@ -6313,7 +6313,7 @@ FFF_END_EXTERN_C SAVE_ARG(FUNCNAME, 16); \ SAVE_ARG(FUNCNAME, 17); \ SAVE_ARG(FUNCNAME, 18); \ - if(ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ + if (ROOM_FOR_MORE_HISTORY(FUNCNAME)){ \ SAVE_ARG_HISTORY(FUNCNAME, 0); \ SAVE_ARG_HISTORY(FUNCNAME, 1); \ SAVE_ARG_HISTORY(FUNCNAME, 2); \ @@ -6358,7 +6358,7 @@ FFF_END_EXTERN_C return FUNCNAME##_fake.custom_fake_seq[FUNCNAME##_fake.custom_fake_seq_len-1](arg0, arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9, arg10, arg11, arg12, arg13, arg14, arg15, arg16, arg17, arg18, ap); \ } \ } \ - if(FUNCNAME##_fake.custom_fake){ \ + if (FUNCNAME##_fake.custom_fake){ \ RETURN_TYPE ret; \ va_list ap; \ va_start(ap, arg18); \ -- 2.7.4 From 697070ec3d5aa383be734646dda576e1ecb7e14b Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Tue, 23 Mar 2021 13:36:20 +0900 Subject: [PATCH 02/16] Update package version to 0.1.73 Change-Id: I4088cca473ab1f7b8771ab2c09547b02e315dd16 Signed-off-by: Jihoon Kim --- packaging/capi-ui-sticker.spec | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 7c39465..2210fec 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -1,6 +1,6 @@ Name: capi-ui-sticker Summary: Sticker client library and daemon -Version: 0.1.72 +Version: 0.1.73 Release: 1 Group: Graphics & UI Framework/Input License: Apache-2.0 @@ -85,7 +85,6 @@ Group: Graphics & UI Framework/Input Sticker client library and daemon (gcov) %endif -%if %{defined _ctest_enable} %package unittests Summary: Sticker tests Group: Development/Libraries @@ -93,7 +92,6 @@ Requires: %{name} = %{version}-%{release} %description unittests GTest for sticker -%endif %prep %setup -q -n %{name}-%{version} @@ -214,10 +212,8 @@ chsmack -a "User::App::Shared" /opt/usr/share/sticker-data %{TZ_SYS_RO_APP}/%{_companion_app_name}/* %endif -%if %{defined _ctest_enable} %files unittests %{_bindir}/* -%endif %if 0%{?gcov:1} %files gcov -- 2.7.4 From 6a078eb2dc55c73d119a2477eb713df0755757a5 Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Wed, 24 Mar 2021 11:24:28 +0900 Subject: [PATCH 03/16] Update version in receiver manifest Change-Id: I33d4b6b1d0420d01a7deefe8a7a435d1e153c7c8 Signed-off-by: Jihoon Kim --- receiver/tizen-manifest.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/receiver/tizen-manifest.xml b/receiver/tizen-manifest.xml index aed506c..cdfdfb9 100644 --- a/receiver/tizen-manifest.xml +++ b/receiver/tizen-manifest.xml @@ -1,5 +1,5 @@ - + -- 2.7.4 From ae4015a8b7c16d936727f60ce852fc3448603116 Mon Sep 17 00:00:00 2001 From: InHong Han Date: Thu, 25 Mar 2021 10:44:45 +0900 Subject: [PATCH 04/16] Change API description by API review Change-Id: I60ef9bf6887c3fb17e10ed43622ba3f765295bbc --- include/sticker_provider.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/sticker_provider.h b/include/sticker_provider.h index 945338d..9690781 100644 --- a/include/sticker_provider.h +++ b/include/sticker_provider.h @@ -202,7 +202,7 @@ int sticker_provider_delete_data(sticker_provider_h provider_handle, sticker_dat /** * @brief Deletes a sticker with the given @a uri from the sticker database. - * @details If the database does not have a sticker that matches the given @a uri, the STICKER_ERROR_NO_SUCH_FILE error is returned. + * @details If the database does not have a sticker that matches the given @a uri, the #STICKER_ERROR_NO_SUCH_FILE error is returned. * @since_tizen 5.5 * @param[in] provider_handle The sticker provider handle * @param[in] uri The URI of the sticker data to be deleted -- 2.7.4 From 91969957ec33581bc7801e0da766f9accfd60892 Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Thu, 25 Mar 2021 11:23:25 +0900 Subject: [PATCH 05/16] Enhance unittest coverage lines: 54.8 (1035 of 1888 lines) -> 66.1% (1308 of 1980 lines) functions..: 73.4% (168 of 229 functions) -> 81.5% (202 of 248 functions) Change-Id: I71f38f23ee8cb02edd3494ffde705bf419c15b55 Signed-off-by: Jihoon Kim --- tests/src/sticker_consumer_unittests.cc | 153 ++++++++++++++++++++++++++++++++ tests/src/sticker_data_unittests.cc | 59 ++++++++++++ tests/src/sticker_provider_unittests.cc | 11 +++ 3 files changed, 223 insertions(+) diff --git a/tests/src/sticker_consumer_unittests.cc b/tests/src/sticker_consumer_unittests.cc index e1b59b4..398ed42 100644 --- a/tests/src/sticker_consumer_unittests.cc +++ b/tests/src/sticker_consumer_unittests.cc @@ -17,6 +17,7 @@ #include #include #include +#include #include #include @@ -29,6 +30,7 @@ static bool callback_result = false; static sticker_data_h g_dh = NULL; +static sticker_provider_h g_ph = NULL; static sticker_consumer_h g_ch = NULL; namespace { @@ -138,6 +140,10 @@ void __fake_g_dbus_connection_signal_unsubscribe(GDBusConnection* c, void __fake_g_bus_unwatch_name(guint a) { } +static void event_cb(sticker_consumer_event_type_e event_type, sticker_data_h data_handle, void *user_data) +{ +} + class StickerConsumerTest : public testing::Test { public: virtual void SetUp() { @@ -172,6 +178,9 @@ class StickerConsumerTest : public testing::Test { sticker_data_set_group_name(g_dh, "tizen"); sticker_data_set_thumbnail(g_dh, "/res/test.png"); sticker_data_set_description(g_dh, "TCT Test"); + sticker_data_set_display_type(g_dh, STICKER_DATA_DISP_WALLPAPER); + sticker_provider_insert_data(g_ph, g_dh); + sticker_provider_set_group_image(g_ph, "tizen", STICKER_DATA_URI_WEB_RESOURCE, "www.valid.com/test/test.png"); } virtual void TearDown() { sticker_data_destroy(g_dh); @@ -258,6 +267,19 @@ TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_all_n) } /** + * @testcase utc_sticker_consumer_data_foreach_by_keyword_p + * @since_tizen 5.5 + * @description Positive UTC of the function that retrieves all sticker data using keyword. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_by_keyword_p) +{ + int result = 0; + callback_result = false; + int ret = sticker_consumer_data_foreach_by_keyword(g_ch, 0, 1000, &result, "cute", sticker_data_foreach_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** * @testcase utc_sticker_consumer_data_foreach_by_keyword_n * @since_tizen 5.5 * @description Negative UTC of the function that retrieves all sticker data using keyword. @@ -270,6 +292,19 @@ TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_by_keyword_n) } /** + * @testcase utc_sticker_consumer_data_foreach_by_group_p + * @since_tizen 5.5 + * @description Positive UTC of the function that retrieves all sticker data using group. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_by_group_p) +{ + int result = 0; + callback_result = false; + int ret = sticker_consumer_data_foreach_by_group(g_ch, 0, 1000, &result, "tizen", sticker_data_foreach_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** * @testcase utc_sticker_consumer_data_foreach_by_group_n * @since_tizen 5.5 * @description Negative UTC of the function that retrieves all sticker data using group. @@ -282,6 +317,19 @@ TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_by_group_n) } /** + * @testcase utc_sticker_consumer_data_foreach_by_type_p + * @since_tizen 5.5 + * @description Positive UTC of the function that retrieves all sticker data using type. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_by_type_p) +{ + int result = 0; + callback_result = false; + int ret = sticker_consumer_data_foreach_by_type(g_ch, 0, 1000, &result, STICKER_DATA_URI_WEB_RESOURCE, sticker_data_foreach_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** * @testcase utc_sticker_consumer_data_foreach_by_type_n * @since_tizen 5.5 * @description Negative UTC of the function that retrieves all sticker data using type. @@ -315,4 +363,109 @@ TEST_F(StickerConsumerTest, utc_sticker_consumer_keyword_list_foreach_all_n) EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); } +/** + * @testcase utc_sticker_consumer_data_foreach_by_display_type_n + * @since_tizen 5.5 + * @description Negative UTC of the function that retrieves all sticker data using display type. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_data_foreach_by_display_type_n) +{ + int result = 0; + int ret = sticker_consumer_data_foreach_by_display_type(g_ch, -1, 0, &result, STICKER_DATA_DISP_WALLPAPER, sticker_data_foreach_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_consumer_group_list_foreach_by_display_type_n + * @since_tizen 5.5 + * @description Negative UTC of the function that retrieves all group name using display type. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_group_list_foreach_by_display_type_n) +{ + int ret = sticker_consumer_group_list_foreach_by_display_type(g_ch, STICKER_DATA_DISP_WALLPAPER, NULL, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_consumer_add_recent_data_n + * @since_tizen 5.5 + * @description Negative UTC of the function that adds entry to recently used stickers list. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_add_recent_data_n) +{ + int ret = sticker_consumer_add_recent_data(g_ch, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_consumer_get_recent_data_list_p + * @since_tizen 5.5 + * @description Positive UTC of the function that gets recently used stickers list. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_get_recent_data_list_p) +{ + int result = 0; + callback_result = false; + sticker_consumer_add_recent_data(g_ch, g_dh); + int ret = sticker_consumer_get_recent_data_list(g_ch, 1, &result, sticker_data_foreach_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** + * @testcase utc_sticker_consumer_get_recent_data_list_n + * @since_tizen 5.5 + * @description Negative UTC of the function that gets recently used stickers list. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_get_recent_data_list_n) +{ + int result = 0; + int ret = sticker_consumer_get_recent_data_list(g_ch, 0, &result, sticker_data_foreach_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_consumer_set_event_cb_p + * @since_tizen 5.5 + * @description Positive UTC of the function that registers a event callback function. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_set_event_cb_p) +{ + callback_result = false; + int ret = sticker_consumer_set_event_cb(g_ch, event_cb, NULL); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** + * @testcase utc_sticker_consumer_set_event_cb_n + * @since_tizen 5.5 + * @description Negative UTC of the function that registers a event callback function. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_set_event_cb_n) +{ + int ret = sticker_consumer_set_event_cb(g_ch, NULL, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_consumer_unset_event_cb_n + * @since_tizen 5.5 + * @description Negative UTC of the function that unregisters a event callback function. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_unset_event_cb_n) +{ + int ret = sticker_consumer_unset_event_cb(NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_consumer_group_image_list_foreach_all_n + * @since_tizen 6.5 + * @description Negative UTC of the function that retrieves images of all sticker groups. + */ +TEST_F(StickerConsumerTest, utc_sticker_consumer_group_image_list_foreach_all_n) +{ + int ret = sticker_consumer_group_image_list_foreach_all(g_ch, NULL, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + } // namespace diff --git a/tests/src/sticker_data_unittests.cc b/tests/src/sticker_data_unittests.cc index 57002ce..31b4468 100644 --- a/tests/src/sticker_data_unittests.cc +++ b/tests/src/sticker_data_unittests.cc @@ -18,6 +18,7 @@ #include #include +#include #include "app_common_mock.h" @@ -500,4 +501,62 @@ TEST_F(StickerDataTest, utc_sticker_data_get_date_n) EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); } +/** + * @testcase utc_sticker_data_set_display_type_p + * @since_tizen 5.5 + * @description Positive UTC of the function that sets display type of sticker data. + */ +TEST_F(StickerDataTest, utc_sticker_data_set_display_type_p) +{ + int ret = sticker_data_set_display_type(g_dh, STICKER_DATA_DISP_EMOJI); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** + * @testcase utc_sticker_data_get_display_type_p + * @since_tizen 5.5 + * @description Positive UTC of the function that gets display type of sticker data. + */ +TEST_F(StickerDataTest, utc_sticker_data_get_display_type_p) +{ + sticker_data_display_type_e disp_type; + int ret = sticker_data_get_display_type(g_dh, &disp_type); + EXPECT_EQ(ret, STICKER_ERROR_NONE); +} + +/** + * @testcase utc_sticker_data_get_display_type_n + * @since_tizen 5.5 + * @description Negative UTC of the function that gets display type of sticker data. + */ +TEST_F(StickerDataTest, utc_sticker_data_get_display_type_n) +{ + int ret = sticker_data_get_display_type(g_dh, NULL); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_data_get_handle_n + * @since_tizen 5.5 + * @description Negative UTC of the function that gets the sticker data handle using URI. + */ +TEST_F(StickerDataTest, tc_sticker_data_get_handle_n) +{ + sticker_data_h tmp_handle = NULL; + int ret = sticker_data_get_handle("", &tmp_handle); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + +/** + * @testcase utc_sticker_data_get_handle_n2 + * @since_tizen 5.5 + * @description Negative UTC of the function that gets the sticker data handle using URI. + */ +TEST_F(StickerDataTest, utc_sticker_data_get_handle_n2) +{ + sticker_data_h tmp_handle = NULL; + int ret = sticker_data_get_handle("www.invalid.com", &tmp_handle); + EXPECT_EQ(ret, STICKER_ERROR_NO_SUCH_FILE); +} + } // namespace diff --git a/tests/src/sticker_provider_unittests.cc b/tests/src/sticker_provider_unittests.cc index 590c48a..98eb73e 100644 --- a/tests/src/sticker_provider_unittests.cc +++ b/tests/src/sticker_provider_unittests.cc @@ -355,4 +355,15 @@ TEST_F(StickerProviderTest, utc_sticker_provider_delete_data_by_uri_n) EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); } +/** + * @testcase utc_sticker_provider_set_group_image_n + * @since_tizen 6.5 + * @description Negative UTC of the function that sets group image. + */ +TEST_F(StickerProviderTest, utc_sticker_provider_set_group_image_n) +{ + int ret = sticker_provider_set_group_image(g_ph, "invalid", STICKER_DATA_URI_WEB_RESOURCE, "www.valid.com/test/image.png"); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); +} + } // namespace -- 2.7.4 From e887881151765cc5d2ae9161156fa14e44ffe8a3 Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Thu, 25 Mar 2021 13:56:08 +0900 Subject: [PATCH 06/16] Fix possible memory leak in unittest Change-Id: I0fee6f5bd53ee58f9f5b4785d958bffedfec86bc Signed-off-by: Jihoon Kim --- tests/src/sticker_data_unittests.cc | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/src/sticker_data_unittests.cc b/tests/src/sticker_data_unittests.cc index 31b4468..5557f29 100644 --- a/tests/src/sticker_data_unittests.cc +++ b/tests/src/sticker_data_unittests.cc @@ -544,6 +544,9 @@ TEST_F(StickerDataTest, tc_sticker_data_get_handle_n) { sticker_data_h tmp_handle = NULL; int ret = sticker_data_get_handle("", &tmp_handle); + if (tmp_handle) + sticker_data_destroy(tmp_handle); + EXPECT_EQ(ret, STICKER_ERROR_INVALID_PARAMETER); } @@ -556,6 +559,9 @@ TEST_F(StickerDataTest, utc_sticker_data_get_handle_n2) { sticker_data_h tmp_handle = NULL; int ret = sticker_data_get_handle("www.invalid.com", &tmp_handle); + if (tmp_handle) + sticker_data_destroy(tmp_handle); + EXPECT_EQ(ret, STICKER_ERROR_NO_SUCH_FILE); } -- 2.7.4 From 21414bdf1da3ec0062c39c450a75e46832b8391d Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Thu, 25 Mar 2021 13:47:21 +0900 Subject: [PATCH 07/16] Fix issue no executable is included in unittest package Change-Id: I165bccf668b16f8e644f0f2892ef72413fd121e5 Signed-off-by: Jihoon Kim --- CMakeLists.txt | 2 +- packaging/capi-ui-sticker.spec | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 916ce15..7da1298 100755 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -55,11 +55,11 @@ ENDIF() ## Test IF("${TEST_ENABLE}" STREQUAL "true") ENABLE_TESTING() +ENDIF() SET(UTC_STICKER sticker_unittests) ADD_TEST(NAME ${UTC_STICKER} COMMAND ${UTC_STICKER} WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}/tests) ADD_SUBDIRECTORY(tests) -ENDIF() ## config ## INSTALL(FILES ${CMAKE_SOURCE_DIR}/capi-ui-sticker.info DESTINATION ${TZ_SYS_RO_SHARE}/parser-plugins) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 2210fec..6855dba 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -213,7 +213,8 @@ chsmack -a "User::App::Shared" /opt/usr/share/sticker-data %endif %files unittests -%{_bindir}/* +%license LICENSE +%{_bindir}/sticker_unittests %if 0%{?gcov:1} %files gcov -- 2.7.4 From 2464ae0869f0d937fd59cb6a9ebd51aa19edde5d Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Thu, 25 Mar 2021 19:36:50 +0900 Subject: [PATCH 08/16] Update package version to 0.1.74 Change-Id: I20efdef65dde92d84af76f45d7556cd2f1a89f14 Signed-off-by: Jihoon Kim --- packaging/capi-ui-sticker.spec | 2 +- receiver/tizen-manifest.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 6855dba..feac92e 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -1,6 +1,6 @@ Name: capi-ui-sticker Summary: Sticker client library and daemon -Version: 0.1.73 +Version: 0.1.74 Release: 1 Group: Graphics & UI Framework/Input License: Apache-2.0 diff --git a/receiver/tizen-manifest.xml b/receiver/tizen-manifest.xml index cdfdfb9..1c2cbb6 100644 --- a/receiver/tizen-manifest.xml +++ b/receiver/tizen-manifest.xml @@ -1,5 +1,5 @@ - + -- 2.7.4 From bdb6702818823a30845b6e13e7af15bae43f6d5f Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Fri, 26 Mar 2021 11:01:19 +0900 Subject: [PATCH 09/16] Remove unused code Unittest Coverage result: lines......: 66.3% (1307 of 1970 lines) functions..: 85.2% (202 of 237 functions) Change-Id: Iec165cf2ea7953e6ae8ef98c9df948c2fad9269e Signed-off-by: Jihoon Kim --- tests/mock/mock.cc | 11 ----------- tests/src/sticker_provider_unittests.cc | 21 --------------------- 2 files changed, 32 deletions(-) diff --git a/tests/mock/mock.cc b/tests/mock/mock.cc index 75be995..de7faf4 100644 --- a/tests/mock/mock.cc +++ b/tests/mock/mock.cc @@ -17,21 +17,10 @@ DEFINE_FAKE_VALUE_FUNC(int, cynara_initialize, cynara**, DEFINE_FAKE_VALUE_FUNC(int, cynara_finish, cynara*); DEFINE_FAKE_VALUE_FUNC(int, cynara_check, cynara*, const char*, const char*, const char*, const char*); -DEFINE_FAKE_VALUE_FUNC(int, cynara_creds_gdbus_get_client, GDBusConnection*, - const gchar*, enum cynara_client_creds, gchar**); -DEFINE_FAKE_VALUE_FUNC(int, cynara_creds_gdbus_get_user, GDBusConnection*, - const gchar*, enum cynara_user_creds, gchar**); /* gio */ DEFINE_FAKE_VALUE_FUNC(GDBusConnection*, g_bus_get_sync, GBusType, GCancellable*, GError**); -DEFINE_FAKE_VALUE_FUNC(guint, g_bus_own_name_on_connection, - GDBusConnection*, const gchar*, GBusNameOwnerFlags, - GBusNameAcquiredCallback, GBusNameLostCallback, gpointer, - GDestroyNotify); -DEFINE_FAKE_VALUE_FUNC(gboolean, g_dbus_connection_emit_signal, - GDBusConnection*, const gchar*, const gchar*, - const gchar*, const gchar*, GVariant*, GError**); DEFINE_FAKE_VALUE_FUNC(guint, g_bus_watch_name_on_connection, GDBusConnection*, const gchar*, GBusNameWatcherFlags, GBusNameAppearedCallback, GBusNameVanishedCallback, gpointer, diff --git a/tests/src/sticker_provider_unittests.cc b/tests/src/sticker_provider_unittests.cc index 98eb73e..2818166 100644 --- a/tests/src/sticker_provider_unittests.cc +++ b/tests/src/sticker_provider_unittests.cc @@ -33,22 +33,6 @@ static bool callback_result = false; static sticker_data_h g_dh = NULL; static sticker_provider_h g_ph = NULL; -// cynara -int __cynara_initialize(cynara** c, const cynara_configuration* conf) { - *c = (void *)0x1; - return 0; -} - -int __cynara_finish(cynara* c) { - return 0; -} - -int __cynara_check(cynara* c, const char* client, const char* client_session, - const char* user, - const char* privilege) { - return CYNARA_API_ACCESS_ALLOWED; -} - // app common int __fake_app_get_id(char** app_id) { if (app_id) @@ -151,11 +135,6 @@ static void foreach_to_check_del(sticker_data_h data_handle, void *user_data) class StickerProviderTest : public testing::Test { public: virtual void SetUp() { - // cynara - cynara_initialize_fake.custom_fake = __cynara_initialize; - cynara_finish_fake.custom_fake = __cynara_finish; - cynara_check_fake.custom_fake = __cynara_check; - // app-common app_get_id_fake.custom_fake = __fake_app_get_id; -- 2.7.4 From 1d20805ce05aa04f26e1016529084d1078513323 Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Wed, 31 Mar 2021 17:38:25 +0900 Subject: [PATCH 10/16] Update package version to 0.1.75 Change-Id: I23de32154aa0fea3554db000841edc4e62b2f118 Signed-off-by: Jihoon Kim --- packaging/capi-ui-sticker.spec | 2 +- receiver/tizen-manifest.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index feac92e..5bfc02c 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -1,6 +1,6 @@ Name: capi-ui-sticker Summary: Sticker client library and daemon -Version: 0.1.74 +Version: 0.1.75 Release: 1 Group: Graphics & UI Framework/Input License: Apache-2.0 diff --git a/receiver/tizen-manifest.xml b/receiver/tizen-manifest.xml index 1c2cbb6..cecaf3e 100644 --- a/receiver/tizen-manifest.xml +++ b/receiver/tizen-manifest.xml @@ -1,5 +1,5 @@ - + -- 2.7.4 From db574706917ed3ffe1ab975d347d73151f5e935e Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Fri, 23 Apr 2021 10:59:00 +0900 Subject: [PATCH 11/16] Fix build error on gcov option Change-Id: I5ec5db55b92a2e6c3fb8e699f0bd7108a4c44066 Signed-off-by: Jihoon Kim --- CMakeLists.txt | 2 -- packaging/capi-ui-sticker.spec | 18 ++++++------------ 2 files changed, 6 insertions(+), 14 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 7da1298..4279bb2 100755 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -53,9 +53,7 @@ ADD_SUBDIRECTORY(receiver) ENDIF() ## Test -IF("${TEST_ENABLE}" STREQUAL "true") ENABLE_TESTING() -ENDIF() SET(UTC_STICKER sticker_unittests) ADD_TEST(NAME ${UTC_STICKER} COMMAND ${UTC_STICKER} WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}/tests) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 5bfc02c..125e876 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -115,12 +115,7 @@ export FFLAGS+=" -DTIZEN_DEBUG_ENABLE -fvisibility=hidden" %endif -DCMAKE_INSTALL_PREFIX=/usr -DLIBDIR=%{_libdir} -DBINDIR=%{_bindir} -DINCLUDEDIR=%{_includedir} \ -DTZ_SYS_RO_SHARE=%TZ_SYS_RO_SHARE -DTZ_SYS_BIN=%TZ_SYS_BIN -DTZ_USER_SHARE=%TZ_USER_SHARE \ - -DTZ_SYS_RO_APP=%TZ_SYS_RO_APP -DTZ_SYS_RO_PACKAGES=%TZ_SYS_RO_PACKAGES \ -%if %{defined _ctest_enable} - -DTEST_ENABLE="%{_ctest_enable}" \ -%else - -DTEST_ENABLE="false" \ -%endif + -DTZ_SYS_RO_APP=%TZ_SYS_RO_APP -DTZ_SYS_RO_PACKAGES=%TZ_SYS_RO_PACKAGES make %{?jobs:-j%jobs} @@ -158,13 +153,12 @@ tar xf %{name}-gcov.tar -C %{buildroot}%{_datadir}/gcov/obj %endif %check -%if "%{_ctest_enable}" == "true" -ctest --output-on-failure %{?_smp_mflags} -%endif - %if 0%{?gcov:1} -lcov -c --ignore-errors graph --no-external -q -d . -o gcov.info -genhtml gcov.info +ctest --output-on-failure %{?_smp_mflags} +lcov -c --ignore-errors graph --no-external -q -d . -o capi-ui-sticker.info +genhtml capi-ui-sticker.info -o capi-ui-sticker.out +zip -r capi-ui-sticker.zip capi-ui-sticker.out capi-ui-sticker.info +install -m 0644 capi-ui-sticker.zip %{buildroot}%{_datadir}/gcov %endif %post -- 2.7.4 From 9aabbc8ac53ef75bf4212a5cfc4b7b0aeb0476ff Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Fri, 23 Apr 2021 11:00:46 +0900 Subject: [PATCH 12/16] Update package version to 0.1.76 Change-Id: Ib232040e342a826e255a362cb46c319edb48fc17 Signed-off-by: Jihoon Kim --- packaging/capi-ui-sticker.spec | 2 +- receiver/tizen-manifest.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 125e876..564e95b 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -1,6 +1,6 @@ Name: capi-ui-sticker Summary: Sticker client library and daemon -Version: 0.1.75 +Version: 0.1.76 Release: 1 Group: Graphics & UI Framework/Input License: Apache-2.0 diff --git a/receiver/tizen-manifest.xml b/receiver/tizen-manifest.xml index cecaf3e..e3ec8e9 100644 --- a/receiver/tizen-manifest.xml +++ b/receiver/tizen-manifest.xml @@ -1,5 +1,5 @@ - + -- 2.7.4 From 1b0b0d5a38fb1ef17688d6545c1970e40b9dd6fd Mon Sep 17 00:00:00 2001 From: InHong Han Date: Fri, 21 May 2021 17:20:41 +0900 Subject: [PATCH 13/16] Fix issue detected by static analysis tool double_free: Calling g_variant_iter_free frees pointer info_iter which has already been freed. pass_freed_arg: Passing freed pointer keyword_iter as an argument to g_variant_iter_free. Change-Id: I9517519228317fbf921f5bef60641146c20ff72d --- client/src/sticker_dbus.c | 25 ------------------------- 1 file changed, 25 deletions(-) diff --git a/client/src/sticker_dbus.c b/client/src/sticker_dbus.c index 75c1a89..a36cfaa 100644 --- a/client/src/sticker_dbus.c +++ b/client/src/sticker_dbus.c @@ -179,13 +179,6 @@ static void _handle_sticker_consumer_cb(GDBusConnection *connection, g_variant_get(parameters, "(ia{iv}a(s))", &event_type, &info_iter, &keyword_iter); _get_sticker_info_from_gvariant(info_iter, keyword_iter, sticker_data); consumer_handle->event_cb((sticker_consumer_event_type_e)event_type, sticker_data, consumer_handle->event_cb_user_data); - - if (info_iter) - g_variant_iter_free(info_iter); - - if (keyword_iter) - g_variant_iter_free(keyword_iter); - sticker_data_destroy(sticker_data); } } @@ -619,12 +612,6 @@ int sticker_dbus_update_sticker_info(GDBusConnection *gdbus_connection, sticker_ g_variant_get(reply_body, "(a{iv}a(s))", &info_iter, &keyword_iter); _get_sticker_info_from_gvariant(info_iter, keyword_iter, origin_data); - - if (info_iter) - g_variant_iter_free(info_iter); - - if (keyword_iter) - g_variant_iter_free(keyword_iter); } else { LOGW("failed to get sticker info"); free(origin_data); @@ -781,12 +768,6 @@ int sticker_dbus_get_sticker_info_by_record_id(GDBusConnection *gdbus_connection if (reply_body) g_variant_unref(reply_body); - - if (info_iter) - g_variant_iter_free(info_iter); - - if (keyword_iter) - g_variant_iter_free(keyword_iter); } if (reply) @@ -1132,12 +1113,6 @@ int sticker_dbus_get_sticker_info_by_uri(GDBusConnection *gdbus_connection, stic if (reply_body) g_variant_unref(reply_body); - - if (info_iter) - g_variant_iter_free(info_iter); - - if (keyword_iter) - g_variant_iter_free(keyword_iter); } if (reply) -- 2.7.4 From 5c47ae67baccdadbe490d894079e2a014dd37e5a Mon Sep 17 00:00:00 2001 From: InHong Han Date: Fri, 21 May 2021 17:22:56 +0900 Subject: [PATCH 14/16] Update package version to 0.1.77 Change-Id: I58d8623a5fd6bf4a28e093ce513f23094e0ce800 --- packaging/capi-ui-sticker.spec | 2 +- receiver/tizen-manifest.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 564e95b..842f64e 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -1,6 +1,6 @@ Name: capi-ui-sticker Summary: Sticker client library and daemon -Version: 0.1.76 +Version: 0.1.77 Release: 1 Group: Graphics & UI Framework/Input License: Apache-2.0 diff --git a/receiver/tizen-manifest.xml b/receiver/tizen-manifest.xml index e3ec8e9..32057c9 100644 --- a/receiver/tizen-manifest.xml +++ b/receiver/tizen-manifest.xml @@ -1,5 +1,5 @@ - + -- 2.7.4 From a53c9df27d5b04334b4b2e6dac044403bdfee40b Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Mon, 6 Sep 2021 09:52:26 +0900 Subject: [PATCH 15/16] Fix issue detected by static analysis tool Change-Id: I44ca1ca13b33e9f6dc014a7700fca22f4f7322b1 Signed-off-by: Jihoon Kim --- server/stickerd_data_manager.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/server/stickerd_data_manager.c b/server/stickerd_data_manager.c index 3210959..ac976f2 100644 --- a/server/stickerd_data_manager.c +++ b/server/stickerd_data_manager.c @@ -862,6 +862,7 @@ int stickerd_insert_sticker_info_by_json(GVariant *parameters, GVariant **reply_ JsonArray *keyword_arr = NULL; int arr_len = 0; int keyword_arr_len = 0; + const char *keyword = NULL; JsonNode *root = NULL; JsonObject *root_obj = NULL; @@ -967,7 +968,9 @@ int stickerd_insert_sticker_info_by_json(GVariant *parameters, GVariant **reply_ goto free_memory; for (int j = 0; j < keyword_arr_len; j++) { - sticker_info->keyword = g_list_append(sticker_info->keyword, strdup((const char *)json_array_get_string_element(keyword_arr, j))); + keyword = json_array_get_string_element(keyword_arr, j); + if (keyword) + sticker_info->keyword = g_list_append(sticker_info->keyword, strdup(keyword)); } ret = stickerd_db_insert_sticker_info(&record_id, sticker_info); @@ -1898,4 +1901,4 @@ int stickerd_get_group_image_list(GVariant *parameters, GVariant **reply_body) } return ret; -} \ No newline at end of file +} -- 2.7.4 From 0b7854c07e132af8df0b8b864fa3e63aa4dd5ead Mon Sep 17 00:00:00 2001 From: Jihoon Kim Date: Mon, 6 Sep 2021 09:55:20 +0900 Subject: [PATCH 16/16] Update package version to 0.1.78 Change-Id: I098e1947801f6bcb6d7c15c2b0527de3409fff33 Signed-off-by: Jihoon Kim --- packaging/capi-ui-sticker.spec | 2 +- receiver/tizen-manifest.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging/capi-ui-sticker.spec b/packaging/capi-ui-sticker.spec index 842f64e..8d2b693 100644 --- a/packaging/capi-ui-sticker.spec +++ b/packaging/capi-ui-sticker.spec @@ -1,6 +1,6 @@ Name: capi-ui-sticker Summary: Sticker client library and daemon -Version: 0.1.77 +Version: 0.1.78 Release: 1 Group: Graphics & UI Framework/Input License: Apache-2.0 diff --git a/receiver/tizen-manifest.xml b/receiver/tizen-manifest.xml index 32057c9..da652b6 100644 --- a/receiver/tizen-manifest.xml +++ b/receiver/tizen-manifest.xml @@ -1,5 +1,5 @@ - + -- 2.7.4