On Thu, 28 Apr 2016 20:33:07 +0200 Benoit Gschwind <[email protected]> wrote:
> I did a patches set that split the previous patch in severals steps. The step > intend to help the review, and may not be logical. The patch is focuced on > keeping the legacy configuration behaviours. > > Benoit Gschwind (9): > wayland-backend: Create a basic config structure to use in new init > API > wayland-backend: fix memory leak of display_name > wayland-backend: use new config struture to create backend > wayland-backend: move config parsing in create_cursor > wayland-backend: define output configuration structure > wayland-backend: split wayland_output_create_for_config > wayland-backend: split init_backend > wayland-backend: put configuration structure in separate header > wayland-backend: move all parsing functions to weston > > Makefile.am | 3 + > src/compositor-wayland.c | 187 +++++++++++------------------------------ > src/compositor-wayland.h | 63 ++++++++++++++ > src/main.c | 211 > ++++++++++++++++++++++++++++++++++++++++++++++- > 4 files changed, 323 insertions(+), 141 deletions(-) > create mode 100644 src/compositor-wayland.h > Hi, I looked through patches 1-8 and I think they are acceptable, with some changes I did: - Patches 5 and 6 squashed plus fixing fullscreen scale value back to the hardcoded 1. - Patch 7 rebased. - Patch 8 needed moving the #include out of the extern "C". Patch 9 is still to review. Push: d62004e..639322a master -> master Thanks, pq
pgpUiFiyoXWWv.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
