Remove xdg-output support

xdg-output was used to extract the output name, which can be obtained
from wl_output as of interface version 4, release in Wayland 1.20 almost
two years ago.
master
Kenny Levinsen 2023-09-01 14:36:29 +02:00
parent 966728a29b
commit c59bbbd6e3
3 changed files with 73 additions and 305 deletions

154
main.c
View File

@ -16,7 +16,6 @@
#include <wayland-client-protocol.h> #include <wayland-client-protocol.h>
#include <wayland-client.h> #include <wayland-client.h>
#include "xdg-output-unstable-v1-client-protocol.h"
#include "wlr-gamma-control-unstable-v1-client-protocol.h" #include "wlr-gamma-control-unstable-v1-client-protocol.h"
#include "color_math.h" #include "color_math.h"
#include "str_vec.h" #include "str_vec.h"
@ -145,7 +144,6 @@ struct context {
enum force_state forced_state; enum force_state forced_state;
struct zwlr_gamma_control_manager_v1 *gamma_control_manager; struct zwlr_gamma_control_manager_v1 *gamma_control_manager;
struct zxdg_output_manager_v1 *xdg_output_manager;
}; };
struct output { struct output {
@ -153,7 +151,6 @@ struct output {
struct context *context; struct context *context;
struct wl_output *wl_output; struct wl_output *wl_output;
struct zxdg_output_v1 *xdg_output;
struct zwlr_gamma_control_v1 *gamma_control; struct zwlr_gamma_control_v1 *gamma_control;
int table_fd; int table_fd;
@ -162,6 +159,7 @@ struct output {
uint16_t *table; uint16_t *table;
bool enabled; bool enabled;
char *name; char *name;
char *description;
}; };
static void print_trajectory(struct context *ctx) { static void print_trajectory(struct context *ctx) {
@ -401,6 +399,62 @@ static void update_timer(const struct context *ctx, timer_t timer, time_t now) {
timer_settime(timer, TIMER_ABSTIME, &timerspec, NULL); timer_settime(timer, TIMER_ABSTIME, &timerspec, NULL);
} }
static void wl_output_handle_geometry(void *data, struct wl_output *output, int x, int y, int width,
int height, int subpixel, const char *make, const char *model,
int transform) {
(void)data, (void)output, (void)x, (void)y, (void)width, (void)height, (void)subpixel,
(void)make, (void)model, (void)transform;
}
static void wl_output_handle_mode(void *data, struct wl_output *output, uint32_t flags, int width,
int height, int refresh) {
(void)data, (void)output, (void)flags, (void)width, (void)height, (void)refresh;
}
static void wl_output_handle_done(void *data, struct wl_output *output) {
(void)data, (void)output;
}
static void wl_output_handle_scale(void *data, struct wl_output *output, int scale) {
(void)data, (void)output, (void)scale;
}
static void wl_output_handle_name(void *data, struct wl_output *wl_output, const char *name) {
(void)wl_output;
struct output *output = data;
output->name = strdup(name);
struct config *cfg = &output->context->config;
for (size_t idx = 0; idx < cfg->output_names.len; ++idx) {
if (strcmp(output->name, cfg->output_names.data[idx]) == 0) {
fprintf(stderr, "enabling output %s by name\n", output->name);
output->enabled = true;
return;
}
}
}
static void wl_output_handle_description(void *data, struct wl_output *wl_output, const char *description) {
(void)wl_output;
struct output *output = data;
output->description = strdup(description);
struct config *cfg = &output->context->config;
for (size_t idx = 0; idx < cfg->output_names.len; ++idx) {
if (strcmp(output->description, cfg->output_names.data[idx]) == 0) {
fprintf(stderr, "enabling output %s by description\n", output->description);
output->enabled = true;
return;
}
}
}
struct wl_output_listener wl_output_listener = {
.geometry = wl_output_handle_geometry,
.mode = wl_output_handle_mode,
.done = wl_output_handle_done,
.scale = wl_output_handle_scale,
.name = wl_output_handle_name,
.description = wl_output_handle_description,
};
static int create_anonymous_file(off_t size) { static int create_anonymous_file(off_t size) {
char template[] = "/tmp/wlsunset-shared-XXXXXX"; char template[] = "/tmp/wlsunset-shared-XXXXXX";
@ -479,64 +533,6 @@ static const struct zwlr_gamma_control_v1_listener gamma_control_listener = {
.failed = gamma_control_handle_failed, .failed = gamma_control_handle_failed,
}; };
static void xdg_output_handle_logical_position(void *data,
struct zxdg_output_v1 *xdg_output, int32_t x, int32_t y) {
(void)data, (void)xdg_output, (void)x, (void)y;
}
static void xdg_output_handle_logical_size(void *data,
struct zxdg_output_v1 *xdg_output, int32_t width, int32_t height) {
(void)data, (void)xdg_output, (void)width, (void)height;
}
static void xdg_output_handle_done(void *data,
struct zxdg_output_v1 *xdg_output) {
(void)data, (void)xdg_output;
}
static void xdg_output_handle_name(void *data,
struct zxdg_output_v1 *xdg_output, const char *name) {
(void)xdg_output;
struct output *output = data;
output->name = strdup(name);
struct config *cfg = &output->context->config;
for (size_t idx = 0; idx < cfg->output_names.len; ++idx) {
if (strcmp(output->name, cfg->output_names.data[idx]) == 0) {
fprintf(stderr, "enabling output %s by name\n", output->name);
output->enabled = true;
return;
}
}
}
static void xdg_output_handle_description(void *data,
struct zxdg_output_v1 *xdg_output, const char *description) {
(void)data, (void)xdg_output, (void)description;
}
static const struct zxdg_output_v1_listener xdg_output_listener = {
.logical_position = xdg_output_handle_logical_position,
.logical_size = xdg_output_handle_logical_size,
.done = xdg_output_handle_done,
.name = xdg_output_handle_name,
.description = xdg_output_handle_description,
};
static void setup_xdg_output(struct context *ctx, struct output *output) {
if (output->xdg_output != NULL) {
return;
}
if (ctx->xdg_output_manager == NULL) {
fprintf(stderr, "skipping setup of output %d: xdg_output_manager is missing\n",
output->id);
return;
}
output->xdg_output = zxdg_output_manager_v1_get_xdg_output(
ctx->xdg_output_manager, output->wl_output);
zxdg_output_v1_add_listener(output->xdg_output, &xdg_output_listener, output);
}
static void setup_gamma_control(struct context *ctx, struct output *output) { static void setup_gamma_control(struct context *ctx, struct output *output) {
if (output->gamma_control != NULL) { if (output->gamma_control != NULL) {
return; return;
@ -558,24 +554,27 @@ static void registry_handle_global(void *data, struct wl_registry *registry,
struct context *ctx = (struct context *)data; struct context *ctx = (struct context *)data;
if (strcmp(interface, wl_output_interface.name) == 0) { if (strcmp(interface, wl_output_interface.name) == 0) {
fprintf(stderr, "registry: adding output %d\n", name); fprintf(stderr, "registry: adding output %d\n", name);
struct output *output = calloc(1, sizeof(struct output)); struct output *output = calloc(1, sizeof(struct output));
output->id = name; output->id = name;
output->wl_output = wl_registry_bind(registry, name,
&wl_output_interface, 1);
output->table_fd = -1; output->table_fd = -1;
output->enabled = true;
output->context = ctx; output->context = ctx;
wl_list_insert(&ctx->outputs, &output->link);
if (ctx->config.output_names.len > 0) { if (version >= WL_OUTPUT_NAME_SINCE_VERSION) {
setup_xdg_output(ctx, output); output->enabled = ctx->config.output_names.len == 0;
output->enabled = false; output->wl_output = wl_registry_bind(registry, name,
&wl_output_interface, WL_OUTPUT_NAME_SINCE_VERSION);
wl_output_add_listener(output->wl_output, &wl_output_listener, output);
} else {
fprintf(stderr, "wl_output: old version (%d < %d), disabling name support\n",
version, WL_OUTPUT_NAME_SINCE_VERSION);
output->enabled = true;
output->wl_output = wl_registry_bind(registry, name,
&wl_output_interface, version);
} }
wl_list_insert(&ctx->outputs, &output->link);
setup_gamma_control(ctx, output); setup_gamma_control(ctx, output);
} else if (ctx->config.output_names.len > 0 && strcmp(interface,
zxdg_output_manager_v1_interface.name) == 0) {
ctx->xdg_output_manager = wl_registry_bind(registry, name,
&zxdg_output_manager_v1_interface, version);
} else if (strcmp(interface, } else if (strcmp(interface,
zwlr_gamma_control_manager_v1_interface.name) == 0) { zwlr_gamma_control_manager_v1_interface.name) == 0) {
ctx->gamma_control_manager = wl_registry_bind(registry, name, ctx->gamma_control_manager = wl_registry_bind(registry, name,
@ -592,9 +591,6 @@ static void registry_handle_global_remove(void *data,
if (output->id == name) { if (output->id == name) {
fprintf(stderr, "registry: removing output %d\n", name); fprintf(stderr, "registry: removing output %d\n", name);
wl_list_remove(&output->link); wl_list_remove(&output->link);
if (output->xdg_output != NULL) {
zxdg_output_v1_destroy(output->xdg_output);
}
if (output->gamma_control != NULL) { if (output->gamma_control != NULL) {
zwlr_gamma_control_v1_destroy(output->gamma_control); zwlr_gamma_control_v1_destroy(output->gamma_control);
} }
@ -801,11 +797,6 @@ static int wlrun(struct config cfg) {
wl_registry_add_listener(registry, &registry_listener, &ctx); wl_registry_add_listener(registry, &registry_listener, &ctx);
wl_display_roundtrip(display); wl_display_roundtrip(display);
if (ctx.config.output_names.len > 0 && ctx.xdg_output_manager == NULL) {
fprintf(stderr, "compositor doesn't support xdg-output-unstable-v1\n");
return EXIT_FAILURE;
}
if (ctx.gamma_control_manager == NULL) { if (ctx.gamma_control_manager == NULL) {
fprintf(stderr, "compositor doesn't support wlr-gamma-control-unstable-v1\n"); fprintf(stderr, "compositor doesn't support wlr-gamma-control-unstable-v1\n");
return EXIT_FAILURE; return EXIT_FAILURE;
@ -813,9 +804,6 @@ static int wlrun(struct config cfg) {
struct output *output; struct output *output;
wl_list_for_each(output, &ctx.outputs, link) { wl_list_for_each(output, &ctx.outputs, link) {
if (ctx.config.output_names.len > 0) {
setup_xdg_output(&ctx, output);
}
setup_gamma_control(&ctx, output); setup_gamma_control(&ctx, output);
} }
wl_display_roundtrip(display); wl_display_roundtrip(display);

View File

@ -35,8 +35,8 @@ scanner = find_program('wayland-scanner')
scanner_private_code = generator(scanner, output: '@BASENAME@-protocol.c', arguments: ['private-code', '@INPUT@', '@OUTPUT@']) scanner_private_code = generator(scanner, output: '@BASENAME@-protocol.c', arguments: ['private-code', '@INPUT@', '@OUTPUT@'])
scanner_client_header = generator(scanner, output: '@BASENAME@-client-protocol.h', arguments: ['client-header', '@INPUT@', '@OUTPUT@']) scanner_client_header = generator(scanner, output: '@BASENAME@-client-protocol.h', arguments: ['client-header', '@INPUT@', '@OUTPUT@'])
protocols_src = [scanner_private_code.process('wlr-gamma-control-unstable-v1.xml', 'xdg-output-unstable-v1.xml')] protocols_src = [scanner_private_code.process('wlr-gamma-control-unstable-v1.xml')]
protocols_headers = [scanner_client_header.process('wlr-gamma-control-unstable-v1.xml', 'xdg-output-unstable-v1.xml')] protocols_headers = [scanner_client_header.process('wlr-gamma-control-unstable-v1.xml')]
wl_client = dependency('wayland-client') wl_client = dependency('wayland-client')
wl_protocols = dependency('wayland-protocols') wl_protocols = dependency('wayland-protocols')

View File

@ -1,220 +0,0 @@
<?xml version="1.0" encoding="UTF-8"?>
<protocol name="xdg_output_unstable_v1">
<copyright>
Copyright © 2017 Red Hat Inc.
Permission is hereby granted, free of charge, to any person obtaining a
copy of this software and associated documentation files (the "Software"),
to deal in the Software without restriction, including without limitation
the rights to use, copy, modify, merge, publish, distribute, sublicense,
and/or sell copies of the Software, and to permit persons to whom the
Software is furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice (including the next
paragraph) shall be included in all copies or substantial portions of the
Software.
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
DEALINGS IN THE SOFTWARE.
</copyright>
<description summary="Protocol to describe output regions">
This protocol aims at describing outputs in a way which is more in line
with the concept of an output on desktop oriented systems.
Some information are more specific to the concept of an output for
a desktop oriented system and may not make sense in other applications,
such as IVI systems for example.
Typically, the global compositor space on a desktop system is made of
a contiguous or overlapping set of rectangular regions.
Some of the information provided in this protocol might be identical
to their counterparts already available from wl_output, in which case
the information provided by this protocol should be preferred to their
equivalent in wl_output. The goal is to move the desktop specific
concepts (such as output location within the global compositor space,
the connector name and types, etc.) out of the core wl_output protocol.
Warning! The protocol described in this file is experimental and
backward incompatible changes may be made. Backward compatible
changes may be added together with the corresponding interface
version bump.
Backward incompatible changes are done by bumping the version
number in the protocol and interface names and resetting the
interface version. Once the protocol is to be declared stable,
the 'z' prefix and the version number in the protocol and
interface names are removed and the interface version number is
reset.
</description>
<interface name="zxdg_output_manager_v1" version="3">
<description summary="manage xdg_output objects">
A global factory interface for xdg_output objects.
</description>
<request name="destroy" type="destructor">
<description summary="destroy the xdg_output_manager object">
Using this request a client can tell the server that it is not
going to use the xdg_output_manager object anymore.
Any objects already created through this instance are not affected.
</description>
</request>
<request name="get_xdg_output">
<description summary="create an xdg output from a wl_output">
This creates a new xdg_output object for the given wl_output.
</description>
<arg name="id" type="new_id" interface="zxdg_output_v1"/>
<arg name="output" type="object" interface="wl_output"/>
</request>
</interface>
<interface name="zxdg_output_v1" version="3">
<description summary="compositor logical output region">
An xdg_output describes part of the compositor geometry.
This typically corresponds to a monitor that displays part of the
compositor space.
For objects version 3 onwards, after all xdg_output properties have been
sent (when the object is created and when properties are updated), a
wl_output.done event is sent. This allows changes to the output
properties to be seen as atomic, even if they happen via multiple events.
</description>
<request name="destroy" type="destructor">
<description summary="destroy the xdg_output object">
Using this request a client can tell the server that it is not
going to use the xdg_output object anymore.
</description>
</request>
<event name="logical_position">
<description summary="position of the output within the global compositor space">
The position event describes the location of the wl_output within
the global compositor space.
The logical_position event is sent after creating an xdg_output
(see xdg_output_manager.get_xdg_output) and whenever the location
of the output changes within the global compositor space.
</description>
<arg name="x" type="int"
summary="x position within the global compositor space"/>
<arg name="y" type="int"
summary="y position within the global compositor space"/>
</event>
<event name="logical_size">
<description summary="size of the output in the global compositor space">
The logical_size event describes the size of the output in the
global compositor space.
For example, a surface without any buffer scale, transformation
nor rotation set, with the size matching the logical_size will
have the same size as the corresponding output when displayed.
Most regular Wayland clients should not pay attention to the
logical size and would rather rely on xdg_shell interfaces.
Some clients such as Xwayland, however, need this to configure
their surfaces in the global compositor space as the compositor
may apply a different scale from what is advertised by the output
scaling property (to achieve fractional scaling, for example).
For example, for a wl_output mode 3840×2160 and a scale factor 2:
- A compositor not scaling the surface buffers will advertise a
logical size of 3840×2160,
- A compositor automatically scaling the surface buffers will
advertise a logical size of 1920×1080,
- A compositor using a fractional scale of 1.5 will advertise a
logical size of 2560×1440.
For example, for a wl_output mode 1920×1080 and a 90 degree rotation,
the compositor will advertise a logical size of 1080x1920.
The logical_size event is sent after creating an xdg_output
(see xdg_output_manager.get_xdg_output) and whenever the logical
size of the output changes, either as a result of a change in the
applied scale or because of a change in the corresponding output
mode(see wl_output.mode) or transform (see wl_output.transform).
</description>
<arg name="width" type="int"
summary="width in global compositor space"/>
<arg name="height" type="int"
summary="height in global compositor space"/>
</event>
<event name="done">
<description summary="all information about the output have been sent">
This event is sent after all other properties of an xdg_output
have been sent.
This allows changes to the xdg_output properties to be seen as
atomic, even if they happen via multiple events.
For objects version 3 onwards, this event is deprecated. Compositors
are not required to send it anymore and must send wl_output.done
instead.
</description>
</event>
<!-- Version 2 additions -->
<event name="name" since="2">
<description summary="name of this output">
Many compositors will assign names to their outputs, show them to the
user, allow them to be configured by name, etc. The client may wish to
know this name as well to offer the user similar behaviors.
The naming convention is compositor defined, but limited to
alphanumeric characters and dashes (-). Each name is unique among all
wl_output globals, but if a wl_output global is destroyed the same name
may be reused later. The names will also remain consistent across
sessions with the same hardware and software configuration.
Examples of names include 'HDMI-A-1', 'WL-1', 'X11-1', etc. However, do
not assume that the name is a reflection of an underlying DRM
connector, X11 connection, etc.
The name event is sent after creating an xdg_output (see
xdg_output_manager.get_xdg_output). This event is only sent once per
xdg_output, and the name does not change over the lifetime of the
wl_output global.
</description>
<arg name="name" type="string" summary="output name"/>
</event>
<event name="description" since="2">
<description summary="human-readable description of this output">
Many compositors can produce human-readable descriptions of their
outputs. The client may wish to know this description as well, to
communicate the user for various purposes.
The description is a UTF-8 string with no convention defined for its
contents. Examples might include 'Foocorp 11" Display' or 'Virtual X11
output via :1'.
The description event is sent after creating an xdg_output (see
xdg_output_manager.get_xdg_output) and whenever the description
changes. The description is optional, and may not be sent at all.
For objects of version 2 and lower, this event is only sent once per
xdg_output, and the description does not change over the lifetime of
the wl_output global.
</description>
<arg name="description" type="string" summary="output description"/>
</event>
</interface>
</protocol>