[PATCH weston 3/6] Move the functions launching clients to main.c

Quentin Glidic sardemff7+wayland at sardemff7.net
Wed May 25 08:15:02 UTC 2016


On 24/05/2016 18:59, Giulio Camuffo wrote:
> They belong in the compositor rather than libweston since they
> set signals handlers, and a library should not do that behind its
> user's back. Besides, they were using functions in main.c already
> so they were not usable by other compositors.
>
> Signed-off-by: Giulio Camuffo <giuliocamuffo at gmail.com>
> ---
>  ivi-shell/hmi-controller.c     |   1 +
>  src/compositor.c               | 144 ----------------------------------------
>  src/compositor.h               |  22 -------
>  src/main.c                     | 145 +++++++++++++++++++++++++++++++++++++++++
>  src/text-backend.c             |   1 +
>  src/weston.h                   |  24 +++++++
>  tests/ivi_layout-test-plugin.c |   1 +
>  tests/weston-test.c            |   1 +
>  xwayland/xwayland.h            |   1 +
>  9 files changed, 174 insertions(+), 166 deletions(-)

Quickly checked it just moves code around.
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Still, it needs weston.h so it cannot land before patch 2 without changes.

Cheers,

-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list