From: Boram Park <boram1288.park@samsung.com>
Date: Fri, 6 Apr 2018 07:42:06 +0000 (+0900)
Subject: buffer: remove unuseful function
X-Git-Tag: accepted/tizen/unified/20180412.073819~6
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Flibtdm.git;a=commitdiff_plain;h=c5d34acb3cd217e269f9215b2fa64d8a7b4b55d6

buffer: remove unuseful function

the behavior of tdm_buffer_remove_release_handler_internal is not clear.
even if commit is failed, the buffer release function seems to need to be called.

Change-Id: Idbf92ea109272d5556680ed87212095126d8b1cf
---

diff --git a/src/tdm_buffer.c b/src/tdm_buffer.c
index 81716b5..97857b4 100644
--- a/src/tdm_buffer.c
+++ b/src/tdm_buffer.c
@@ -170,26 +170,6 @@ tdm_buffer_remove_release_handler(tbm_surface_h buffer,
 	}
 }
 
-INTERN void
-tdm_buffer_remove_release_handler_internal(tbm_surface_h buffer)
-{
-	tdm_buffer_info *buf_info;
-	tdm_buffer_func_info *func_info = NULL, *next = NULL;
-
-	TDM_RETURN_IF_FAIL(buffer != NULL);
-
-	buf_info = tdm_buffer_get_info(buffer);
-	TDM_RETURN_IF_FAIL(buf_info != NULL);
-
-	LIST_FOR_EACH_ENTRY_SAFE(func_info, next, &buf_info->release_funcs, link) {
-
-		LIST_DEL(&func_info->link);
-		free(func_info);
-
-		return;
-	}
-}
-
 EXTERN tbm_surface_h
 tdm_buffer_ref_backend(tbm_surface_h buffer)
 {
diff --git a/src/tdm_capture.c b/src/tdm_capture.c
index 4b58fbd..1fd5bb4 100644
--- a/src/tdm_capture.c
+++ b/src/tdm_capture.c
@@ -663,12 +663,6 @@ tdm_capture_commit(tdm_capture *capture)
 
 		if (ret != TDM_ERROR_NONE) {
 			/* LCOV_EXCL_START */
-			/* Not to call the user release handler when failed.
-			 * Do we have to call this function here really?
-			 * User better use set_done_handler to know when pp is done. Using
-			 * buffer_release_handler is not good.
-			 */
-			tdm_buffer_remove_release_handler_internal(b->buffer);
 
 			_pthread_mutex_unlock(&private_display->lock);
 			tdm_buffer_unref_backend(b->buffer);
diff --git a/src/tdm_pp.c b/src/tdm_pp.c
index 1a0e91e..a53531a 100644
--- a/src/tdm_pp.c
+++ b/src/tdm_pp.c
@@ -573,14 +573,6 @@ tdm_pp_commit(tdm_pp *pp)
 			continue;
 
 		if (ret != TDM_ERROR_NONE) {
-			/* Not to call the user release handler when failed.
-			 * Do we have to call this function here really?
-			 * User better use set_done_handler to know when pp is done. Using
-			 * buffer_release_handler is not good.
-			 */
-			tdm_buffer_remove_release_handler_internal(b->src);
-			tdm_buffer_remove_release_handler_internal(b->dst);
-
 			_pthread_mutex_unlock(&private_display->lock);
 			tdm_buffer_unref_backend(b->src);
 			tdm_buffer_unref_backend(b->dst);
diff --git a/src/tdm_private.h b/src/tdm_private.h
index 40f9e79..22c984c 100644
--- a/src/tdm_private.h
+++ b/src/tdm_private.h
@@ -187,8 +187,6 @@ tdm_capture_destroy_internal(tdm_private_capture *private_capture);
 /* utility buffer functions for private */
 tdm_buffer_info*
 tdm_buffer_get_info(tbm_surface_h buffer);
-void
-tdm_buffer_remove_release_handler_internal(tbm_surface_h buffer);
 
 /* event functions for private */
 tdm_error