From: antognolli <antognolli>
Date: Tue, 6 Sep 2011 21:15:50 +0000 (+0000)
Subject: emotion/generic: Non-blocking track info retrieval.
X-Git-Tag: submit/2.0alpha-wayland/20121127.222018~139
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=efdc477b3a001b59d81d4e5de0e79796fc180030;p=profile%2Fivi%2Femotion.git

emotion/generic: Non-blocking track info retrieval.

git-svn-id: http://svn.enlightenment.org/svn/e/trunk/emotion@63237 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33
---

diff --git a/src/generic_players/vlc/emotion_generic_vlc.c b/src/generic_players/vlc/emotion_generic_vlc.c
index 8143128..c4ab25b 100644
--- a/src/generic_players/vlc/emotion_generic_vlc.c
+++ b/src/generic_players/vlc/emotion_generic_vlc.c
@@ -693,7 +693,7 @@ _position_changed(struct _App *app)
    _send_resize(app, w, h);
 
    /* sending audio track info */
-   // _send_all_track_info(app);
+   _send_all_track_info(app);
 
    /* sending meta info */
    // _send_all_meta_info(app);
diff --git a/src/modules/generic/emotion_generic.c b/src/modules/generic/emotion_generic.c
index a9f175c..fc44657 100644
--- a/src/modules/generic/emotion_generic.c
+++ b/src/modules/generic/emotion_generic.c
@@ -416,20 +416,6 @@ _player_seekable_changed(Emotion_Generic_Video *ev)
 }
 
 static void
-_player_volume(Emotion_Generic_Video *ev)
-{
-   float vol, oldvol;
-   _player_float_read(ev, &vol);
-
-   INF("received volume: %0.3f", vol);
-
-   oldvol = ev->volume;
-   ev->volume = vol;
-   if (vol != oldvol && !ev->opening)
-     _emotion_audio_level_change(ev->obj);
-}
-
-static void
 _audio_channels_free(Emotion_Generic_Video *ev)
 {
    int i;
@@ -465,20 +451,14 @@ _player_tracks_info(Emotion_Generic_Video *ev, Emotion_Generic_Channel **channel
    Emotion_Generic_Channel *pchannels;
    int i;
 
-   _player_int_read(ev, current);
-   _player_int_read(ev, count);
+   *count = ev->cmd.param.track.total;
+   *current = ev->cmd.param.track.current;
+   pchannels = ev->cmd.param.track.channels;
 
    INF("number of tracks: %d (current = %d):", *count, *current);
-   pchannels = calloc(*count, sizeof(Emotion_Generic_Channel));
    for (i = 0; i < *count; i++)
      {
-	int tid, len;
-	char buf[PATH_MAX];
-	_player_int_read(ev, &tid);
-	_player_str_read(ev, buf, &len);
-	pchannels[i].id = tid;
-	pchannels[i].name = eina_stringshare_add_length(buf, len);
-	INF("\tchannel %d: %s", tid, buf);
+	INF("\tchannel %d: %s", pchannels[i].id, pchannels[i].name);
      }
 
    *channels = pchannels;
@@ -725,14 +705,82 @@ _player_cmd_double_int_process(Emotion_Generic_Video *ev)
 }
 
 static void
+_player_cmd_track_info(Emotion_Generic_Video *ev)
+{
+   int param;
+   Eina_Bool r;
+   int i;
+
+   if (ev->cmd.num_params == 0)
+     {
+	ev->cmd.cur_param = 0;
+	ev->cmd.num_params = 2;
+	ev->cmd.param.track.channels = NULL;
+	ev->cmd.s_len = -1;
+     }
+
+   while (ev->cmd.cur_param < 2)
+     {
+	if (!_player_cmd_param_read(ev, &param, sizeof(param)))
+	  return;
+
+	if (ev->cmd.cur_param == 0)
+	  ev->cmd.param.track.current = param;
+	else
+	  {
+	     ev->cmd.param.track.total = param;
+	     ev->cmd.num_params += param * 2;
+	     ev->cmd.param.track.channels =
+		calloc(param, sizeof(*ev->cmd.param.track.channels));
+	  }
+	ev->cmd.cur_param++;
+     }
+
+   if (ev->cmd.cur_param == ev->cmd.num_params)
+     {
+	_player_cmd_process(ev);
+	return;
+     }
+
+   i = (ev->cmd.cur_param - 2) / 2;
+   if ((ev->cmd.cur_param % 2) == 0) // reading track id
+     {
+	if (!_player_cmd_param_read(ev, &param, sizeof(param)))
+	  return;
+	ev->cmd.param.track.channels[i].id = param;
+	ev->cmd.cur_param++;
+     }
+   else // reading track name
+     {
+	char buf[PATH_MAX];
+
+	if (ev->cmd.s_len == -1)
+	  {
+	     if (!_player_cmd_param_read(ev, &param, sizeof(param)))
+	       return;
+	     ev->cmd.s_len = param;
+	  }
+
+	if (!_player_cmd_param_read(ev, buf, ev->cmd.s_len))
+	  return;
+	ev->cmd.param.track.channels[i].name = 
+	   eina_stringshare_add_length(buf, ev->cmd.s_len);
+	ev->cmd.cur_param++;
+	ev->cmd.s_len = -1;
+     }
+
+   if (ev->cmd.cur_param == ev->cmd.num_params)
+     _player_cmd_process(ev);
+}
+
+static void
 _player_cmd_read(Emotion_Generic_Video *ev)
 {
    if (ev->cmd.type < 0)
      {
 	if (!_player_cmd_param_read(ev, &ev->cmd.type, sizeof(ev->cmd.type)))
 	  return;
-	else
-	  ev->cmd.num_params = 0;
+	ev->cmd.num_params = 0;
      }
 
    switch (ev->cmd.type) {
@@ -754,6 +802,11 @@ _player_cmd_read(Emotion_Generic_Video *ev)
       case EM_RESULT_FRAME_SIZE:
 	 _player_cmd_double_int_process(ev);
 	 break;
+      case EM_RESULT_AUDIO_TRACK_INFO:
+      case EM_RESULT_VIDEO_TRACK_INFO:
+      case EM_RESULT_SPU_TRACK_INFO:
+	 _player_cmd_track_info(ev);
+	 break;
 
       default:
 	 WRN("received wrong command: %d", ev->cmd.type);
diff --git a/src/modules/generic/emotion_generic.h b/src/modules/generic/emotion_generic.h
index 04006b2..38e7c42 100644
--- a/src/modules/generic/emotion_generic.h
+++ b/src/modules/generic/emotion_generic.h
@@ -39,6 +39,7 @@ struct _Emotion_Generic_Cmd_Buffer
    char *tmp;
    int type;
    ssize_t i, total;
+   int s_len;
    int num_params, cur_param;
    int padding;
    union {
@@ -48,6 +49,11 @@ struct _Emotion_Generic_Cmd_Buffer
 	} size;
 	int i_num;
 	float f_num;
+	struct {
+	     int total;
+	     int current;
+	     Emotion_Generic_Channel *channels;
+	} track;
    } param;
 };
 
@@ -90,13 +96,13 @@ struct _Emotion_Generic_Video
    Eina_Bool		     file_ready : 1;
    int			     audio_channels_count;
    int			     audio_channel_current;
-   struct _Emotion_Generic_Channel *audio_channels;
+   Emotion_Generic_Channel  *audio_channels;
    int			     video_channels_count;
    int			     video_channel_current;
-   struct _Emotion_Generic_Channel *video_channels;
+   Emotion_Generic_Channel  *video_channels;
    int			     spu_channels_count;
    int			     spu_channel_current;
-   struct _Emotion_Generic_Channel *spu_channels;
+   Emotion_Generic_Channel  *spu_channels;
    Emotion_Generic_Meta	     meta;
 };