From: Ji-hoon Lee Date: Tue, 17 Sep 2019 07:57:55 +0000 (+0900) Subject: Fix incorrect log messages X-Git-Tag: submit/tizen/20190920.111209~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F27%2F214027%2F5;p=platform%2Fcore%2Fuifw%2Fmulti-assistant.git Fix incorrect log messages Change-Id: I9af54d1180bce387a9b17607db2a78f60eff3266 --- diff --git a/client/ma.c b/client/ma.c index 6e4ff98..d00c49f 100644 --- a/client/ma.c +++ b/client/ma.c @@ -1092,7 +1092,7 @@ int ma_send_recognition_result(ma_recognition_result_event_e result) return MA_ERROR_INVALID_STATE; } - SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send result to the Multi-assistant"); + SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send recognition result to the Multi-assistant"); int ret = -1; int pid = getpid(); @@ -1221,7 +1221,7 @@ int ma_stop_receiving_audio_streaming_data(ma_audio_streaming_data_type_e type) return MA_ERROR_INVALID_STATE; } - SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send error to the Multi-assistant"); + SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send streaming stop request to the Multi-assistant"); int ret = -1; int pid = getpid(); @@ -1257,15 +1257,15 @@ int ma_update_voice_feedback_state(ma_voice_feedback_state_e feedback_state) return MA_ERROR_INVALID_STATE; } - SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send error to the Multi-assistant"); + SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Update voice feedback state to the Multi-assistant"); int ret = -1; int pid = getpid(); ret = ma_dbus_update_voice_feedback_state(pid, feedback_state); if (0 != ret) { - SLOG(LOG_ERROR, TAG_MAC, "[ERROR] Fail to send result"); + SLOG(LOG_ERROR, TAG_MAC, "[ERROR] Fail to update voice feedback state"); } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] Success to send result"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] Success to update voice feedback state"); } return ret; @@ -1293,15 +1293,15 @@ int ma_send_assistant_specific_command(const char* command) return MA_ERROR_INVALID_STATE; } - SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send error to the Multi-assistant"); + SLOG(LOG_DEBUG, TAG_MAC, "[Client DEBUG] Send assistant specific command to the Multi-assistant"); int ret = -1; int pid = getpid(); ret = ma_dbus_send_assistant_specific_command(pid, command); if (0 != ret) { - SLOG(LOG_ERROR, TAG_MAC, "[ERROR] Fail to send result"); + SLOG(LOG_ERROR, TAG_MAC, "[ERROR] Fail to send assistant specific command"); } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] Success to send result"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] Success to send assistant specific command"); } return ret; @@ -1504,7 +1504,7 @@ int ma_get_recording_audio_source_type(char** type) { int ma_set_background_volume(double ratio) { - SLOG(LOG_DEBUG, TAG_MAC, "[Manager] Adjust background volume"); + SLOG(LOG_DEBUG, TAG_MAC, "[Manager] Set background volume"); if (0 != __ma_get_feature_enabled()) { SLOG(LOG_DEBUG, TAG_MAC, "@@@ [Manager] not supported"); @@ -1533,7 +1533,7 @@ int ma_set_background_volume(double ratio) int pid = getpid(); int ret = ma_dbus_set_background_volume(pid, ratio); if (0 != ret) { - SLOG(LOG_WARN, TAG_MAC, "[WARNING] retry to change volume"); + SLOG(LOG_WARN, TAG_MAC, "[WARNING] retry to set background volume"); } else { SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] Success to send"); } @@ -1571,7 +1571,7 @@ int ma_set_preprocessing_allow_mode(ma_preprocessing_allow_mode_e mode, const ch int pid = getpid(); int ret = ma_dbus_set_preprocessing_allow_mode(pid, mode, app_id); if (0 != ret) { - SLOG(LOG_WARN, TAG_MAC, "[WARNING] retry to change volume"); + SLOG(LOG_WARN, TAG_MAC, "[WARNING] retry to set preprocessing allow mode"); } else { SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] Success to send"); } diff --git a/client/ma_dbus.c b/client/ma_dbus.c index 69f268e..611050d 100644 --- a/client/ma_dbus.c +++ b/client/ma_dbus.c @@ -991,10 +991,10 @@ int ma_dbus_stop_streaming_audio_data(int pid, ma_audio_streaming_data_type_e ty MA_METHOD_STOP_STREAMING_AUDIO_DATA); if (NULL == msg) { - SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant send recognition result : Fail to make message"); //LCOV_EXCL_LINE + SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant stop streaming audio data : Fail to make message"); //LCOV_EXCL_LINE return MA_ERROR_OPERATION_FAILED; } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant send recognition result"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant stop streaming audio data"); } dbus_message_append_args(msg, @@ -1039,10 +1039,10 @@ int ma_dbus_update_voice_feedback_state(int pid, ma_voice_feedback_state_e state MA_METHOD_UPDATE_VOICE_FEEDBACK_STATE); if (NULL == msg) { - SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant send recognition result : Fail to make message"); //LCOV_EXCL_LINE + SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant update voice feedback state : Fail to make message"); //LCOV_EXCL_LINE return MA_ERROR_OPERATION_FAILED; } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant send recognition result"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant update voice feedback state"); } dbus_message_append_args(msg, @@ -1087,10 +1087,10 @@ int ma_dbus_send_assistant_specific_command(int pid, const char* command) MA_METHOD_SEND_ASSISTANT_SPECIFIC_COMMAND); if (NULL == msg) { - SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant send recognition result : Fail to make message"); //LCOV_EXCL_LINE + SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant send assistant specific command : Fail to make message"); //LCOV_EXCL_LINE return MA_ERROR_OPERATION_FAILED; } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant send recognition result"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant send assistant specific command"); } char* tmp_command = NULL; @@ -1153,10 +1153,10 @@ int ma_dbus_set_background_volume(int pid, double ratio) MA_METHOD_SET_BACKGROUND_VOLUME); if (NULL == msg) { - SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant change system volume : Fail to make message"); //LCOV_EXCL_LINE + SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant set background system volume : Fail to make message"); //LCOV_EXCL_LINE return MA_ERROR_OPERATION_FAILED; } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant change system volume"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant set background system volume"); } dbus_message_append_args(msg, @@ -1202,10 +1202,10 @@ int ma_dbus_set_preprocessing_allow_mode(int pid, ma_preprocessing_allow_mode_e MA_METHOD_SET_PREPROCESSING_ALLOW_MODE); if (NULL == msg) { - SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant change system volume : Fail to make message"); //LCOV_EXCL_LINE + SLOG(LOG_ERROR, TAG_MAC, "@@ Request multi-assistant set preprocessing allow mode : Fail to make message"); //LCOV_EXCL_LINE return MA_ERROR_OPERATION_FAILED; } else { - SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant change system volume"); + SLOG(LOG_DEBUG, TAG_MAC, "[DEBUG] multi-assistant set preprocessing allow mode"); } char* tmp_app_id = NULL;