Thanks for taking this on. I'm disappointed about the duplication, but if somebody else does the work, I can't really complain.
The whole series is: Reviewed-by: Jasper St. Pierre <[email protected]> On Wed, Oct 1, 2014 at 7:50 AM, Pekka Paalanen <[email protected]> wrote: > From: Pekka Paalanen <[email protected]> > > Hi, > > this is my attempt on the wl_surface role conflict (trying > to change a surface role) error patches. Jasper's original > version is here: > http://lists.freedesktop.org/archives/wayland-devel/2014-August/016810.html > http://lists.freedesktop.org/archives/wayland-devel/2014-August/016811.html > > The main differences are noted in the "compositor:" patch. > > > Thanks, > pq > > Pekka Paalanen (1): > protocol: define error codes for role reassignment > > protocol/wayland.xml | 22 +++++++++++++++++++--- > 1 file changed, 19 insertions(+), 3 deletions(-) > > Pekka Paalanen (3): > protocol: add role error code to xdg_shell > compositor: send error for surface role resets > tests: add role conflict sub-surface vs. wl_shell_surface > > Makefile.am | 5 ++ > desktop-shell/shell.c | 38 ++++--------- > protocol/xdg-shell.xml | 9 +++ > src/compositor.c | 33 +++++++++-- > src/compositor.h | 15 +++++ > src/data-device.c | 11 ++-- > src/input.c | 11 ++-- > tests/roles-test.c | 139 > ++++++++++++++++++++++++++++++++++++++++++++++ > xwayland/window-manager.c | 7 +++ > 9 files changed, 222 insertions(+), 46 deletions(-) > create mode 100644 tests/roles-test.c > > -- > 1.8.5.5 > > -- Jasper
_______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
