[PATCH weston 09/14 v3] weston: Port X11 backend to new output handling API
Pekka Paalanen
ppaalanen at gmail.com
Thu Sep 29 10:17:33 UTC 2016
On Thu, 18 Aug 2016 18:42:37 +0200
Armin Krezović <krezovic.armin at gmail.com> wrote:
> This is a complete port of the X11 backend that
> uses recently added output handling API for output
> configuration.
>
> - Output can be configured at runtime by passing the
> necessary configuration parameters, which can be
> filled in manually, obtained from the configuration
> file or obtained from the command line using
> previously added functionality. It is required that
> the scale and transform values are set using the
> previously added functionality.
>
> - Output can be created at runtime using the output
> API. The output creation only creates a pending
> output, which needs to be configured the same way as
> mentioned above.
>
> Same as before, a single output is created at runtime
> using the default configuration or a configuration
> parsed from the command line. The output-count
> functionality is also preserved, which means more than
> one output can be created initially, and more outputs can
> be added at runtime using the output API.
>
> v2:
>
> - Fix wet_configure_windowed_output_from_config() usage.
> - Call x11_output_disable() explicitly from
> x11_output_destroy().
>
> v3:
>
> - Remove unneeded free().
> - Disallow calling x11_output_configure more than once.
> - Remove unneeded checks for output->name == NULL as that
> has been disallowed.
> - Use weston_compositor_add_pending_output().
> - Bump weston_x11_backend_config version to 2.
>
> Signed-off-by: Armin Krezović <krezovic.armin at gmail.com>
> ---
> compositor/main.c | 151 +++++++++-------------
> libweston/compositor-x11.c | 312 +++++++++++++++++++++++++--------------------
> libweston/compositor-x11.h | 13 +-
> 3 files changed, 235 insertions(+), 241 deletions(-)
Hi,
nice work!
Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> --- a/libweston/compositor-x11.c
> +++ b/libweston/compositor-x11.c
> @@ -59,6 +59,7 @@
> #include "pixman-renderer.h"
> #include "presentation-time-server-protocol.h"
> #include "linux-dmabuf.h"
> +#include "windowed-output-api.h"
>
> #define DEFAULT_AXIS_STEP_DISTANCE 10
>
> @@ -75,6 +76,8 @@ struct x11_backend {
> struct xkb_keymap *xkb_keymap;
> unsigned int has_xkb;
> uint8_t xkb_event_base;
> + int fullscreen;
> + int no_input;
> int use_pixman;
>
> int has_net_wm_state_fullscreen;
Insignificant nitpicks:
- boolean flags should usually be 'bool'
- negative flags often lead to double-negatives in conditions
(!no_input) so for readability would prefer positive flags instead
Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160929/19f5b872/attachment.sig>
More information about the wayland-devel
mailing list