remove unnecessary comment & codes

Change-Id: Ib1e96067e804b7cac3e2cf08f420b0b9ff83c967

diff --git a/src/mm_player_es.c b/src/mm_player_es.c
index a6f2b7b124bc019dd756a29132abb6f3dd810ab0..c9df062bd5f1980360f3e01a3f9216c5471faf18 100644 (file)
--- a/src/mm_player_es.c
+++ b/src/mm_player_es.c
@@ -195,9 +195,6 @@ _mmplayer_set_media_stream_buffer_status_cb(MMHandleType hplayer,
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
-       if ((type < MM_PLAYER_STREAM_TYPE_DEFAULT) || (type > MM_PLAYER_STREAM_TYPE_TEXT))
-               return MM_ERROR_INVALID_ARGUMENT;
-
        MMPLAYER_MEDIA_STREAM_CALLBACK_LOCK(player);
 
        if (player->media_stream_buffer_status_cb[type]) {
@@ -231,8 +228,6 @@ _mmplayer_set_media_stream_seek_data_cb(MMHandleType hplayer,
 
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
 
-       if ((type < MM_PLAYER_STREAM_TYPE_DEFAULT) || (type > MM_PLAYER_STREAM_TYPE_TEXT))
-               return MM_ERROR_INVALID_ARGUMENT;
        MMPLAYER_MEDIA_STREAM_CALLBACK_LOCK(player);
 
        if (player->media_stream_seek_data_cb[type]) {
@@ -318,9 +313,6 @@ _mmplayer_get_media_stream_max_size(MMHandleType hplayer, MMPlayerStreamType typ
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
        MMPLAYER_RETURN_VAL_IF_FAIL(max_size, MM_ERROR_INVALID_ARGUMENT);
 
-       if ((type < MM_PLAYER_STREAM_TYPE_DEFAULT) || (type > MM_PLAYER_STREAM_TYPE_TEXT))
-               return MM_ERROR_INVALID_ARGUMENT;
-
        *max_size = player->media_stream_buffer_max_size[type];
 
        MMPLAYER_FLEAVE();
@@ -366,9 +358,6 @@ _mmplayer_get_media_stream_min_percent(MMHandleType hplayer, MMPlayerStreamType
        MMPLAYER_RETURN_VAL_IF_FAIL(player, MM_ERROR_PLAYER_NOT_INITIALIZED);
        MMPLAYER_RETURN_VAL_IF_FAIL(min_percent, MM_ERROR_INVALID_ARGUMENT);
 
-       if ((type < MM_PLAYER_STREAM_TYPE_DEFAULT) || (type > MM_PLAYER_STREAM_TYPE_TEXT))
-               return MM_ERROR_INVALID_ARGUMENT;
-
        *min_percent = player->media_stream_buffer_min_percent[type];
 
        MMPLAYER_FLEAVE();

diff --git a/src/mm_player_priv.c b/src/mm_player_priv.c
index 43bcc2b63f9229ff3e3ff93738a53faa53de8153..ee4a12229240d9d7ac83849a93e9acf800116e56 100644 (file)
--- a/src/mm_player_priv.c
+++ b/src/mm_player_priv.c
@@ -609,11 +609,6 @@ __mmplayer_set_state(mm_player_t* player, int state)
        MMPLAYER_PREV_STATE(player) = MMPLAYER_CURRENT_STATE(player);
        MMPLAYER_CURRENT_STATE(player) = state;
 
-       /* FIXIT : it's better to do like below code
-       if (MMPLAYER_CURRENT_STATE(player) == MMPLAYER_TARGET_STATE(player))
-                       MMPLAYER_PENDING_STATE(player) = MM_PLAYER_STATE_NONE;
-       and add more code to handling PENDING_STATE.
-       */
        if (MMPLAYER_CURRENT_STATE(player) == MMPLAYER_PENDING_STATE(player))
                MMPLAYER_PENDING_STATE(player) = MM_PLAYER_STATE_NONE;