From: SooChan Lim <sc1.lim@samsung.com>
Date: Fri, 23 Jun 2017 12:49:32 +0000 (+0900)
Subject: e_plane: refine the e_plane_commit_aquire code.
X-Git-Tag: accepted/tizen/unified/20170703.064027~13
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F53%2F135653%2F1;p=platform%2Fupstream%2Fenlightenment.git

e_plane: refine the e_plane_commit_aquire code.

Change-Id: I0b3ce9f7062d59b2569f8dc72af22e7354bedd75
---

diff --git a/src/bin/e_plane.c b/src/bin/e_plane.c
index 68c04b2..c5ebb24 100644
--- a/src/bin/e_plane.c
+++ b/src/bin/e_plane.c
@@ -969,36 +969,15 @@ e_plane_commit_data_aquire(E_Plane *plane)
 
         /* reset to be the initail unset values */
         _e_plane_unset_reset(plane);
-
-        plane->pending_commit_data_list = eina_list_append(plane->pending_commit_data_list, data);
-
-        return data;
      }
-
-   if (!plane->renderer) return NULL;
-
-   /* check update_exist */
-   if (!e_plane_renderer_update_exist_check(plane->renderer))
-     return NULL;
-
-   if (plane->is_fb && !plane->ec)
+   else
      {
-        data = E_NEW(E_Plane_Commit_Data, 1);
-        data->plane = plane;
-        data->renderer = plane->renderer;
-        data->tsurface = plane->tsurface;
-        tbm_surface_internal_ref(data->tsurface);
-        data->ec = NULL;
-
-        plane->pending_commit_data_list = eina_list_append(plane->pending_commit_data_list, data);
+        if (!plane->renderer) return NULL;
 
-        /* set the update_exist to be false */
-        e_plane_renderer_update_exist_set(plane->renderer, EINA_FALSE);
+        /* check update_exist */
+        if (!e_plane_renderer_update_exist_check(plane->renderer))
+          return NULL;
 
-        return data;
-     }
-   else
-     {
         if (plane->ec)
           {
              data = E_NEW(E_Plane_Commit_Data, 1);
@@ -1007,18 +986,33 @@ e_plane_commit_data_aquire(E_Plane *plane)
              data->tsurface = plane->tsurface;
              tbm_surface_internal_ref(data->tsurface);
              data->ec = plane->ec;
-             e_comp_wl_buffer_reference(&data->buffer_ref, _get_comp_wl_buffer(plane->ec));
-
-             plane->pending_commit_data_list = eina_list_append(plane->pending_commit_data_list, data);
 
              /* set the update_exist to be false */
              e_plane_renderer_update_exist_set(plane->renderer, EINA_FALSE);
 
-             return data;
+             e_comp_wl_buffer_reference(&data->buffer_ref, _get_comp_wl_buffer(plane->ec));
+          }
+        else
+          {
+             if (plane->is_fb)
+               {
+                  data = E_NEW(E_Plane_Commit_Data, 1);
+                  data->plane = plane;
+                  data->renderer = plane->renderer;
+                  data->tsurface = plane->tsurface;
+                  tbm_surface_internal_ref(data->tsurface);
+                  data->ec = NULL;
+
+                  /* set the update_exist to be false */
+                  e_plane_renderer_update_exist_set(plane->renderer, EINA_FALSE);
+               }
           }
      }
 
-   return NULL;
+   if (data)
+     plane->pending_commit_data_list = eina_list_append(plane->pending_commit_data_list, data);
+
+   return data;
 }
 
 EINTERN void