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 /examples/rotation.c | |
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 'examples/rotation.c')
-rw-r--r-- | examples/rotation.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/examples/rotation.c b/examples/rotation.c index c7bd85ae..2770529f 100644 --- a/examples/rotation.c +++ b/examples/rotation.c @@ -59,7 +59,8 @@ static void output_frame_notify(struct wl_listener *listener, void *data) { int32_t width, height; wlr_output_effective_resolution(wlr_output, &width, &height); - struct wlr_output_state output_state = {0}; + struct wlr_output_state output_state; + wlr_output_state_init(&output_state); struct wlr_render_pass *pass = wlr_output_begin_render_pass(wlr_output, &output_state, NULL, NULL); wlr_render_pass_add_rect(pass, &(struct wlr_render_rect_options){ @@ -122,7 +123,8 @@ static void new_output_notify(struct wl_listener *listener, void *data) { sample_output->x_offs = sample_output->y_offs = 0; sample_output->x_vel = sample_output->y_vel = 128; - struct wlr_output_state state = {0}; + struct wlr_output_state state; + wlr_output_state_init(&state); wlr_output_state_set_transform(&state, sample->transform); sample_output->output = output; |