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(....); Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev