diff options
author | Sebastian Lipponer <mail@sebastianlipponer.de> | 2019-11-25 08:22:11 +0100 |
---|---|---|
committer | Saya Sugiura <39760799+ssugiura@users.noreply.github.com> | 2019-11-25 16:22:11 +0900 |
commit | cd040a4ce1ea3d2008048083a862461596cd2cd2 (patch) | |
tree | 9962ef9fd7ff7627322f34dbfd83614b7d8e6ec9 /src/daemon/dlt_daemon_client.c | |
parent | 346781ff633aee48eee5700a0b379d97cf86c935 (diff) | |
download | DLT-daemon-cd040a4ce1ea3d2008048083a862461596cd2cd2.tar.gz |
Change the DLT_CHECK_RCV_DATA_SIZE macro to an internal function (#191)
The use of this macro in a if condition is actually non-standard C++
code. GCC issues a warning that ISO C++ forbids braced-groups within
expressions when compiled with -Wpedantic. Other compilers fail with an
error. Change the macro to an internal function in order to improve the
portability to other platforms (i.e. MSVC, proprietary embedded).
Signed-off-by: Sebastian Lipponer <mail@sebastianlipponer.de>
Diffstat (limited to 'src/daemon/dlt_daemon_client.c')
-rw-r--r-- | src/daemon/dlt_daemon_client.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/daemon/dlt_daemon_client.c b/src/daemon/dlt_daemon_client.c index eefed04..acd46f4 100644 --- a/src/daemon/dlt_daemon_client.c +++ b/src/daemon/dlt_daemon_client.c @@ -881,7 +881,7 @@ void dlt_daemon_control_get_log_info(int sock, if ((daemon == NULL) || (msg == NULL) || (msg->databuffer == NULL)) return; - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceGetLogInfoRequest)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceGetLogInfoRequest)) < 0) return; user_list = dlt_daemon_find_users_list(daemon, daemon->ecuid, verbose); @@ -1740,7 +1740,7 @@ void dlt_daemon_control_set_log_level(int sock, if ((daemon == NULL) || (msg == NULL) || (msg->databuffer == NULL)) return; - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetLogLevel)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetLogLevel)) < 0) return; req = (DltServiceSetLogLevel *)(msg->databuffer); @@ -1913,7 +1913,7 @@ void dlt_daemon_control_set_trace_status(int sock, if ((daemon == NULL) || (msg == NULL) || (msg->databuffer == NULL)) return; - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetLogLevel)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetLogLevel)) < 0) return; req = (DltServiceSetLogLevel *)(msg->databuffer); @@ -2007,7 +2007,7 @@ void dlt_daemon_control_set_default_log_level(int sock, if ((daemon == NULL) || (msg == NULL) || (msg->databuffer == NULL)) return; - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) return; req = (DltServiceSetDefaultLogLevel *)(msg->databuffer); @@ -2047,7 +2047,7 @@ void dlt_daemon_control_set_all_log_level(int sock, return; } - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) return; req = (DltServiceSetDefaultLogLevel *)(msg->databuffer); @@ -2084,7 +2084,7 @@ void dlt_daemon_control_set_default_trace_status(int sock, if ((daemon == NULL) || (msg == NULL) || (msg->databuffer == NULL)) return; - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) return; req = (DltServiceSetDefaultLogLevel *)(msg->databuffer); @@ -2124,7 +2124,7 @@ void dlt_daemon_control_set_all_trace_status(int sock, return; } - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetDefaultLogLevel)) < 0) return; req = (DltServiceSetDefaultLogLevel *)(msg->databuffer); @@ -2161,7 +2161,7 @@ void dlt_daemon_control_set_timing_packets(int sock, if ((daemon == NULL) || (msg == NULL) || (msg->databuffer == NULL)) return; - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceSetVerboseMode)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceSetVerboseMode)) < 0) return; req = (DltServiceSetVerboseMode *)(msg->databuffer); @@ -2429,7 +2429,7 @@ void dlt_daemon_control_service_logstorage(int sock, return; } - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServiceOfflineLogstorage)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServiceOfflineLogstorage)) < 0) return; req = (DltServiceOfflineLogstorage *)(msg->databuffer); @@ -2610,7 +2610,7 @@ void dlt_daemon_control_passive_node_connect(int sock, return; } - if (DLT_CHECK_RCV_DATA_SIZE(msg->datasize, sizeof(DltServicePassiveNodeConnect)) < 0) + if (dlt_check_rcv_data_size(msg->datasize, sizeof(DltServicePassiveNodeConnect)) < 0) return; req = (DltServicePassiveNodeConnect *)msg->databuffer; |