remove unused codes

Change-Id: Ib9ea41906c404e0044478dd1f1df1129d277cc52
Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>

diff --git a/include/playermgr.h b/include/playermgr.h
index 9812f43..90dc6c6 100644 (file)
--- a/include/playermgr.h
+++ b/include/playermgr.h
@@ -52,10 +52,6 @@ int playermgr_get_position(struct playermgr *mgr, int *position);
 int playermgr_set_position(struct playermgr *mgr,
                int sec, player_seek_completed_cb cb, void *data);
 
-int playermgr_set_playback_rate(struct playermgr *mgr, float speed);
-
-int playermgr_get_playback_rate(struct playermgr *mgr, float *speed);
-
 int playermgr_set_ff(struct playermgr *mgr, int duration);
 
 int playermgr_set_rew(struct playermgr *mgr, int duration);
@@ -64,8 +60,6 @@ bool playermgr_get_ffrew(struct playermgr *mgr);
 
 void playermgr_stop_ffrew(struct playermgr *mgr);
 
-int playermgr_set_looping(struct playermgr *mgr, bool value);
-
 int playermgr_set_audio_effect(struct playermgr *mgr, int value);
 
 int playermgr_set_display_mode(struct playermgr *mgr, int value);

diff --git a/src/playermgr.cpp b/src/playermgr.cpp
index 2dbb52c..c4f307a 100644 (file)
--- a/src/playermgr.cpp
+++ b/src/playermgr.cpp
@@ -40,7 +40,6 @@ struct playermgr {
        Evas_Object *win;
        Eina_List *media_list;
 
-       float speed;
        int display_mode;
        int audio_effect;
        int current;
@@ -373,61 +372,6 @@ int playermgr_set_position(struct playermgr *mgr,
        return 0;
 }
 
-int playermgr_set_playback_rate(struct playermgr *mgr, float speed)
-{
-       int r;
-
-       if (!mgr || !mgr->player) {
-               _ERR("invalid parameter");
-               return -1;
-       }
-
-       if (speed > RATE_MAX)
-               speed = RATE_MAX;
-       else if (speed < RATE_MIN)
-               speed = RATE_MIN;
-
-       r = player_set_playback_rate(mgr->player, speed);
-       if (r != PLAYER_ERROR_NONE) {
-               _ERR("Player set playback error");
-               return -1;
-       }
-
-       mgr->speed = speed;
-
-       return 0;
-}
-
-int playermgr_get_playback_rate(struct playermgr *mgr, float *speed)
-{
-       if (!mgr) {
-               _ERR("invalid parameter");
-               return -1;
-       }
-
-       *speed = mgr->speed;
-
-       return 0;
-}
-
-int playermgr_set_looping(struct playermgr *mgr, bool value)
-{
-       int r;
-
-       if (!mgr || !mgr->player) {
-               _ERR("invalid parameter");
-               return -1;
-       }
-
-       r = player_set_looping(mgr->player, value);
-       if (r != PLAYER_ERROR_NONE) {
-               _ERR("Player set looping error");
-               return -1;
-       }
-
-       return 0;
-}
-
 int playermgr_set_audio_effect(struct playermgr *mgr, int value)
 {
        /* audio_effect not supported, just return 0 */

diff --git a/src/videodata.cpp b/src/videodata.cpp
index 1dd053c..74661e6 100644 (file)
--- a/src/videodata.cpp
+++ b/src/videodata.cpp
@@ -46,34 +46,33 @@ bool CVideoData::Update(media_info_h media_h)
        r = video_meta_get_duration(video_h, &duration);
        if (r != MEDIA_CONTENT_ERROR_NONE) {
                _ERR("video meta get duration error");
-               video_meta_destroy(video_h);
-               return false;
+               goto err;
        }
 
        r = video_meta_get_width(video_h, &width);
        if (r != MEDIA_CONTENT_ERROR_NONE) {
                _ERR("video meta get width error");
-               video_meta_destroy(video_h);
-               return false;
+               goto err;
        }
 
        r = video_meta_get_height(video_h, &height);
        if (r != MEDIA_CONTENT_ERROR_NONE) {
                _ERR("video meta get width error");
-               video_meta_destroy(video_h);
-               return false;
+               goto err;
        }
 
        r = video_meta_get_played_position(video_h, &position);
        if (r != MEDIA_CONTENT_ERROR_NONE) {
                _ERR("video meta get played position error");
-               video_meta_destroy(video_h);
-               return false;
+               goto err;
        }
 
        video_meta_destroy(video_h);
-
        return true;
+
+err:
+       video_meta_destroy(video_h);
+       return false;
 }
 
 int CVideoData::GetDuration(void)

diff --git a/src/view_player.cpp b/src/view_player.cpp
index d828a71..806cf48 100644 (file)
--- a/src/view_player.cpp
+++ b/src/view_player.cpp
@@ -594,14 +594,12 @@ bool CPlayerView::Create(void *data)
        r = m_UiInit();
        if (!r) {
                _ERR("failed to init UI");
-               m_Fini();
-               return false;
+               goto err;
        }
 
        if (!m_PlayerInit()) {
                _ERR("failed to play video");
-               m_Fini();
-               return false;
+               goto err;
        }
 
        playermgr_set_completed_cb(m->player, sm_CbPlayComplete, m);
@@ -612,13 +610,16 @@ bool CPlayerView::Create(void *data)
        m->drawanim_timer = ecore_timer_add(0.8, sm_CbDrawAnimation, m);
        if (!m->drawanim_timer) {
                _ERR("failed to add draw timer");
-               m_Fini();
-               return false;
+               goto err;
        }
 
        CBaseView::Create(NULL);
 
        return true;
+
+err:
+       m_Fini();
+       return false;
 }
 
 void CPlayerView::m_Fini(void)