webrtc_test_espp: Remove unnecessary function call 63/290263/1
authorSangchul Lee <sc11.lee@samsung.com>
Wed, 22 Mar 2023 07:07:15 +0000 (16:07 +0900)
committerSangchul Lee <sc11.lee@samsung.com>
Wed, 22 Mar 2023 07:08:47 +0000 (16:08 +0900)
Codes to call esplusplayer_activate() are removed.
Some variable names are changed.

[Version] 0.4.5
[Issue Type] Improvement

Change-Id: Id3d7f1c841b4e155a2a0cfa0e18033fc3c8d2bec
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
packaging/capi-media-webrtc.spec
test/webrtc_test_espp.c
test/webrtc_test_priv.h

index 22bb7be1ecb931b1e0ad4c2ba8c007ddffb64fab..f49b87a5148912a9eb921cd265a4633032b2f0bf 100644 (file)
@@ -1,6 +1,6 @@
 Name:       capi-media-webrtc
 Summary:    A WebRTC library in Tizen Native API
-Version:    0.4.4
+Version:    0.4.5
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
index 2a886e94bc6f3cd7c76a200cc7f56ce24d8c5228..5e9853a1e77aaafbba41f32399c0e072fcf99e50 100644 (file)
@@ -47,8 +47,8 @@ void _espp_init(int index)
                !get_appdata()->conns[index].encoded_video_frame_cb_is_set)
                return;
 
-       get_appdata()->conns[index].render.espp.audio_track_activated = false;
-       get_appdata()->conns[index].render.espp.video_track_activated = false;
+       get_appdata()->conns[index].render.espp.audio_stream_info_set = false;
+       get_appdata()->conns[index].render.espp.video_stream_info_set = false;
        get_appdata()->conns[index].render.espp.audio_ready_to_prepare = false;
        get_appdata()->conns[index].render.espp.video_ready_to_prepare = false;
 
@@ -70,8 +70,8 @@ void _espp_deinit(int index)
        esplusplayer_destroy(get_appdata()->conns[index].render.espp.handle);
 
        get_appdata()->conns[index].render.espp.handle = NULL;
-       get_appdata()->conns[index].render.espp.audio_track_activated = false;
-       get_appdata()->conns[index].render.espp.video_track_activated = false;
+       get_appdata()->conns[index].render.espp.audio_stream_info_set = false;
+       get_appdata()->conns[index].render.espp.video_stream_info_set = false;
        get_appdata()->conns[index].render.espp.audio_ready_to_prepare = false;
        get_appdata()->conns[index].render.espp.video_ready_to_prepare = false;
 }
@@ -197,9 +197,9 @@ espp_info_s espp_infos[] = {
 bool _espp_activate_and_prepare_async(connection_s *conn, media_packet_h packet, webrtc_media_type_e type)
 {
        bool is_audio = (type == WEBRTC_MEDIA_TYPE_AUDIO) ? true : false;
-       bool *track_activated = is_audio ? &conn->render.espp.audio_track_activated : &conn->render.espp.video_track_activated;
-       bool *opposite_track_activated = is_audio ? &conn->render.espp.video_track_activated : &conn->render.espp.audio_track_activated;
-       bool *opposite_type_exist = is_audio ? &conn->encoded_video_frame_cb_is_set : &conn->encoded_audio_frame_cb_is_set;
+       bool *is_stream_info_set = is_audio ? &conn->render.espp.audio_stream_info_set : &conn->render.espp.video_stream_info_set;
+       bool opposite_type_stream_info_set = is_audio ? conn->render.espp.video_stream_info_set : conn->render.espp.audio_stream_info_set;
+       bool opposite_type_exist = is_audio ? conn->encoded_video_frame_cb_is_set : conn->encoded_audio_frame_cb_is_set;
 
        if (!conn)
                return false;
@@ -209,7 +209,7 @@ bool _espp_activate_and_prepare_async(connection_s *conn, media_packet_h packet,
                g_printerr("render.espp.handle is NULL\n");
                return false;
        }
-       if (*track_activated)
+       if (*is_stream_info_set)
                return false;
 
        /* Get format and set espp audiostream & prepare async */
@@ -218,11 +218,9 @@ bool _espp_activate_and_prepare_async(connection_s *conn, media_packet_h packet,
                (void *)&conn->render.espp.audio_info :
                (void *)&conn->render.espp.video_info);
 
-       g_print("espp activate stream_type[%u]\n", espp_infos[type].stream_type);
-       esplusplayer_activate(conn->render.espp.handle, espp_infos[type].stream_type);
-       *track_activated = true;
+       *is_stream_info_set = true;
 
-       if (!*opposite_type_exist || *opposite_track_activated) {
+       if (!opposite_type_exist || opposite_type_stream_info_set) {
                g_print("espp prepare async\n");
                esplusplayer_prepare_async(conn->render.espp.handle);
        }
index b608d4b5aae317883f55611a9668da947a39029a..163544f83cba15d3fe64df7078b8705049d17feb 100644 (file)
@@ -238,8 +238,8 @@ typedef struct _connection_s {
                        esplusplayer_handle handle;
                        esplusplayer_audio_stream_info audio_info;
                        esplusplayer_video_stream_info video_info;
-                       bool audio_track_activated;
-                       bool video_track_activated;
+                       bool audio_stream_info_set;
+                       bool video_stream_info_set;
                        bool audio_ready_to_prepare;
                        bool video_ready_to_prepare;
                } espp;