Hi Emil, Emil Velikov <emil.l.veli...@gmail.com> writes:
> On 20 October 2017 at 12:35, Harish Krupo <harish.krupo....@intel.com> wrote: > >> + else if (!n_rects || !try_damage_buffer(dri2_surf, rects, n_rects)) >> wl_surface_damage(dri2_surf->wl_surface_wrapper, >> 0, 0, INT32_MAX, INT32_MAX); >> > Please add a comment why SetDamageRegionCalled is honoured in the > SwapBuffers case, while it's not for SwapBuffersWithDamage(...NULL, > 0); > Both functions should behave identically although one could interpret > the KHR_partial_update spec in different ways. > > Also swap the hunk with: > > else dri2_wl_set_damage_region(....); > Oops! Sent the wrong v2, sorry. Anyways, I will send v3 with the changes suggested by Eric. Thanks and regards Harish krupo > Thanks > Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
