mainnika/weston
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

weston: store weston_compositor in wet_compositor

Browse Source 
This makes it easier to just pass wet_compositor around and take the
weston_compositor from it.

It feels weird to go from weston_compositor to wet_compositor all the
time in internal functions. It's necessary in callbacks that cannot
carry wet_compositor, but otherwise it is awkward.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Acked-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
This commit is contained in:
Pekka Paalanen
2017-11-02 15:38:24 +02:00
parent dfc0683ba0
commit 3ffc6876ca
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compositor/main.c
+8 -5
View File
@@ -77,6 +77,7 @@ struct wet_head_tracker {
};
struct wet_compositor {
struct weston_compositor *compositor;
struct weston_config *config;
struct wet_output_config *parsed_options;
bool drm_use_current_mode;
@@ -1089,9 +1090,8 @@ wet_head_tracker_create(struct wet_compositor *compositor,
}
static void
simple_head_enable(struct weston_compositor *compositor, struct weston_head *head)
simple_head_enable(struct wet_compositor *wet, struct weston_head *head)
{
struct wet_compositor *wet = to_wet_compositor(compositor);
struct weston_output *output;
int ret = 0;
@@ -1101,7 +1101,8 @@ simple_head_enable(struct weston_compositor *compositor, struct weston_head *hea
if (weston_head_get_output(head))
return;
output = weston_compositor_create_output_with_head(compositor, head);
output = weston_compositor_create_output_with_head(wet->compositor,
head);
if (!output) {
weston_log("Could not create an output for head \"%s\".\n",
weston_head_get_name(head));
@@ -1158,18 +1159,19 @@ static void
simple_heads_changed(struct wl_listener *listener, void *arg)
{
struct weston_compositor *compositor = arg;
struct wet_compositor *wet = to_wet_compositor(compositor);
struct weston_head *head = NULL;
bool connected;
bool enabled;
bool changed;
while ((head = weston_compositor_iterate_heads(compositor, head))) {
while ((head = weston_compositor_iterate_heads(wet->compositor, head))) {
connected = weston_head_is_connected(head);
enabled = weston_head_is_enabled(head);
changed = weston_head_is_device_changed(head);
if (connected && !enabled) {
simple_head_enable(compositor, head);
simple_head_enable(wet, head);
} else if (!connected && enabled) {
simple_head_disable(head);
} else if (enabled && changed) {
@@ -1926,6 +1928,7 @@ int main(int argc, char *argv[])
}
ec = weston_compositor_create(display, &user_data);
user_data.compositor = ec;
if (ec == NULL) {
weston_log("fatal: failed to create compositor\n");
goto out;