Merge "library: Simplify subsession_event_wait_done() error path" into tizen
authorKarol Lewandowski <k.lewandowsk@samsung.com>
Fri, 19 Aug 2022 16:17:56 +0000 (16:17 +0000)
committerGerrit Code Review <gerrit@review>
Fri, 19 Aug 2022 16:17:56 +0000 (16:17 +0000)
libsessiond/src/lib.c

index 08894ae..b0027b9 100644 (file)
@@ -804,10 +804,12 @@ static int subsession_switch_wait_done(int session_uid, uint64_t switch_id) {
        return_with_log_error_result_(method_call_sync_simple(dbus_method_call.SwitchUserDone, params));
 }
 
-EXPORT_API int subsession_event_wait_done(subsession_event_info info) {
+EXPORT_API int subsession_event_wait_done(subsession_event_info info)
+{
        return_if(
                session_uid_is_not_valid(info.session_uid)
        )
+
        switch (info.event) {
                case SUBSESSION_ADD_USER_WAIT:
                        return_if(
@@ -829,13 +831,9 @@ EXPORT_API int subsession_event_wait_done(subsession_event_info info) {
                        )
                        return subsession_switch_wait_done(info.session_uid, info.switch_user.switch_id);
                        break;
-
-               default:
-                       {
-                               ret_if_err_( SUBSESSION_ERROR_INVALID_PARAMETER)
-                       }
        }
-       return_with_log_error_result_( SUBSESSION_ERROR_INVALID_PARAMETER)
+
+       return SUBSESSION_ERROR_INVALID_PARAMETER;
 }
 
 EXPORT_API int subsession_get_user_list(int session_uid, subsession_user_t **user_list, int *user_count)