diff options
| author | Simon Ser <contact@emersion.fr> | 2023-06-22 15:48:35 +0200 |
|---|---|---|
| committer | Alexander Orzechowski <alex@ozal.ski> | 2023-06-23 18:07:26 +0000 |
| commit | be050979684ae8779a56e4b0166cfb21d1935d35 (patch) | |
| tree | c4a77ff1363875abb59508b3b3a5cdff28dbb895 /tinywl | |
| parent | 8a5b5e6f28dae650df36a271213655b8bdf52dbf (diff) | |
output: add wlr_output_state_init()
This changes the semantics of wlr_output_state. Instead of having
fields with uninitialized memory when missing from the committed
bitflag, all fields are always initialized (and maybe NULL/empty),
just like we do in wlr_surface_state. This reduces the chances of
footguns when reading a field, and removes the need to check for
the committed bitfield everywhere.
A new wlr_output_state_init() function takes care of initializing
the Pixman region.
Diffstat (limited to 'tinywl')
| -rw-r--r-- | tinywl/tinywl.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tinywl/tinywl.c b/tinywl/tinywl.c index e6e3ceee..80f08e70 100644 --- a/tinywl/tinywl.c +++ b/tinywl/tinywl.c @@ -603,7 +603,8 @@ static void server_new_output(struct wl_listener *listener, void *data) { * would let the user configure it. */ struct wlr_output_mode *mode = wlr_output_preferred_mode(wlr_output); if (mode != NULL) { - struct wlr_output_state state = {0}; + struct wlr_output_state state; + wlr_output_state_init(&state); wlr_output_state_set_mode(&state, mode); wlr_output_state_set_enabled(&state, true); if (!wlr_output_commit_state(wlr_output, &state)) { |