[PATCH weston 0/2] Unify the commit logic

Jason Ekstrand jason at jlekstrand.net
Fri Jun 20 17:33:02 PDT 2014


It's a code cleanup.  Every time we change the commit logic (see the last
weston commit) we have to do it in two places.  They are bound to get
out-of-sync.
--Jason


On Fri, Jun 20, 2014 at 5:28 PM, Jasper St. Pierre <jstpierre at mecheye.net>
wrote:

> Does this fix any bugs that you know of, or is it just a code cleanup?
> Just curious, I'm not insinuating anything.
>
>
> On Fri, Jun 20, 2014 at 8:25 PM, Jason Ekstrand <jason at jlekstrand.net>
> wrote:
>
>> This little two-patch series unifies all of the surface commit logic in
>> one
>> place.  Previously the commit logic was duplicated in weston_surface
>> and weston_subsurface functions in order to handle the commit cache.
>>
>> Jason Ekstrand (2):
>>   Move weston_subsurface to after weston_surface
>>   Create a weston_surface_state structure for storing pending surface
>>     state     and move the surface commit logic into
>>     weston_surface_commit_state
>>
>>  src/compositor.c | 151
>> +++++++++++++++++++------------------------------------
>>  src/compositor.h | 132 +++++++++++++++++++++---------------------------
>>  2 files changed, 107 insertions(+), 176 deletions(-)
>>
>> --
>> 2.0.0
>>
>> _______________________________________________
>> wayland-devel mailing list
>> wayland-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>>
>
>
>
> --
>   Jasper
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20140620/c41ecfe0/attachment.html>


More information about the wayland-devel mailing list