On Fri, Feb 14, 2014 at 09:31:45AM +0200, Pekka Paalanen wrote:
On Thu, 13 Feb 2014 18:18:23 +0000 "Yeh, Sinclair" sinclair.yeh@intel.com wrote:
The below seems fine, but I wonder if we could make this one cause an error to be returned later where we can, rather than silently ignoring. I'm not sure where or how, though.
Would it make sense to change wl_egl_window_resize() so that it return a value? Either that, or it should be documented somewhere in the API spec that setting width/height <=0 will be ignored.
I'm not sure we can change the function signature, it's public stable ABI.
Surely drivers have maximum size limits, too, those must be catched somewhere already.
But this might be worth looking into: if the window system produces a bad size, what do drivers do when they cannot allocate or render to it?
In X11 it's all hidden from the app, but I don't think the gfx stack can guarantee valid sizes in all cases, can it?
Anyway, my suggestion is just for convenience, and if drivers already just silently do whatever on a bad size, being silent here does not make it any worse.
We could maybe make it raise an EGL error, I'm a little concerned with how this silently fails. However, the behavior with Sinclairs patch is better than what we had before, so I've pushed that.
Thanks, pq
egl_window->width = width; egl_window->height = height; egl_window->dx = dx; @@ -24,6 +27,9 @@ wl_egl_window_create(struct wl_surface *surface, { struct wl_egl_window *egl_window;
- if (width <= 0 || height <= 0)
return NULL;- egl_window = malloc(sizeof *egl_window); if (!egl_window) return NULL;
Thanks, pq _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel
wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel