Re: [PATCH xserver 0/2] xwayland: couple of EGL backend API cleanup

2018-06-15 Thread Olivier Fourdan
Hi, On Fri, Jun 15, 2018 at 9:46 AM, Olivier Fourdan wrote: > Hi Emil, > > On Thu, Jun 14, 2018 at 7:38 PM, Emil Velikov > wrote: >> What's the blocker of pushing these - short on commit access, other? >> If the former, that should be addressed. You have done some serious >> work on Xwayland.

Re: [PATCH xserver 0/2] xwayland: couple of EGL backend API cleanup

2018-06-15 Thread Olivier Fourdan
Hi Emil, On Thu, Jun 14, 2018 at 7:38 PM, Emil Velikov wrote: > What's the blocker of pushing these - short on commit access, other? > If the former, that should be addressed. You have done some serious > work on Xwayland. I don't think I have commit access myself, never tried actually, never

Re: [PATCH xserver 0/2] xwayland: couple of EGL backend API cleanup

2018-06-14 Thread Emil Velikov
On 11 June 2018 at 09:22, Olivier Fourdan wrote: > Hi, > > The recent issue with Present due to the wrong pixmap size being given > to xwl_glamor_pixmap_get_wl_buffer() got me thinking. > > Considering that the code always use the drawable size now, that passing > a wrong size is error prone

[PATCH xserver 0/2] xwayland: couple of EGL backend API cleanup

2018-06-11 Thread Olivier Fourdan
Hi, The recent issue with Present due to the wrong pixmap size being given to xwl_glamor_pixmap_get_wl_buffer() got me thinking. Considering that the code always use the drawable size now, that passing a wrong size is error prone because once the buffer is created for the given pixmap,