tbm_surface_internal: clean up tbm_surface_internal_is_valid() 11/94811/1
authorYoungJun Cho <yj44.cho@samsung.com>
Tue, 1 Nov 2016 09:03:21 +0000 (18:03 +0900)
committerYoungJun Cho <yj44.cho@samsung.com>
Tue, 1 Nov 2016 09:18:30 +0000 (18:18 +0900)
This patch cleans up tbm_surface_internal_is_valid().
It is enough to use LIST_FOR_EACH_ENTRY because there is no deletion routine.

Change-Id: I58e63e9f3de99ec3195df19385d3a675fb3a32f3
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>
src/tbm_surface_internal.c

index 71dd8aa..974902d 100644 (file)
@@ -344,7 +344,7 @@ _tbm_surface_internal_destroy(tbm_surface_h surface)
 int
 tbm_surface_internal_is_valid(tbm_surface_h surface)
 {
-       tbm_surface_h old_data = NULL, tmp = NULL;
+       tbm_surface_h old_data = NULL;
 
        if (surface == NULL || g_surface_bufmgr == NULL) {
                TBM_TRACE("error: tbm_surface(%p)\n", surface);
@@ -352,7 +352,7 @@ tbm_surface_internal_is_valid(tbm_surface_h surface)
        }
 
        if (!LIST_IS_EMPTY(&g_surface_bufmgr->surf_list)) {
-               LIST_FOR_EACH_ENTRY_SAFE(old_data, tmp, &g_surface_bufmgr->surf_list, item_link) {
+               LIST_FOR_EACH_ENTRY(old_data, &g_surface_bufmgr->surf_list, item_link) {
                        if (old_data == surface) {
                                TBM_TRACE("tbm_surface(%p)\n", surface);
                                return 1;