From e9bf4efa713959e68d61db7fbca93f7dfea2ae5e Mon Sep 17 00:00:00 2001 From: infirit Date: Wed, 9 Jul 2014 00:52:37 +0200 Subject: Adapt to MateDesktop API changes MateRRScreen, MateRRConfig and MateRROutputInfo (formerly MateOutputInfo) have turned into opaque GObjects. Use accessors for modifying them, and allocate/free them as GObjects. Based on gnome-control-center commit: 3eb3ca4506624fb537c55796d1397eb5f9b8da4b From: Giovanni Campagna --- capplets/display/xrandr-capplet.c | 443 +++++++++++++++++++------------------- 1 file changed, 226 insertions(+), 217 deletions(-) (limited to 'capplets/display') diff --git a/capplets/display/xrandr-capplet.c b/capplets/display/xrandr-capplet.c index 2460028a..103fb34f 100644 --- a/capplets/display/xrandr-capplet.c +++ b/capplets/display/xrandr-capplet.c @@ -49,7 +49,7 @@ struct App MateRRScreen *screen; MateRRConfig *current_configuration; MateRRLabeler *labeler; - MateOutputInfo *current_output; + MateRROutputInfo *current_output; GtkWidget *dialog; GtkWidget *current_monitor_event_box; @@ -91,13 +91,13 @@ enum { static void rebuild_gui (App *app); static void on_clone_changed (GtkWidget *box, gpointer data); static void on_rate_changed (GtkComboBox *box, gpointer data); -static gboolean output_overlaps (MateOutputInfo *output, MateRRConfig *config); +static gboolean output_overlaps (MateRROutputInfo *output, MateRRConfig *config); static void select_current_output_from_dialog_position (App *app); static void monitor_on_off_toggled_cb (GtkToggleButton *toggle, gpointer data); -static void get_geometry (MateOutputInfo *output, int *w, int *h); +static void get_geometry (MateRROutputInfo *output, int *w, int *h); static void apply_configuration_returned_cb (DBusGProxy *proxy, DBusGProxyCall *call_id, void *data); static gboolean get_clone_size (MateRRScreen *screen, int *width, int *height); -static gboolean output_info_supports_mode (App *app, MateOutputInfo *info, int width, int height); +static gboolean output_info_supports_mode (App *app, MateRROutputInfo *info, int width, int height); static void error_message (App *app, const char *primary_text, const char *secondary_text) @@ -139,10 +139,10 @@ on_screen_changed (MateRRScreen *scr, MateRRConfig *current; App *app = data; - current = mate_rr_config_new_current (app->screen); + current = mate_rr_config_new_current (app->screen, NULL); if (app->current_configuration) - mate_rr_config_free (app->current_configuration); + g_object_unref (app->current_configuration); app->current_configuration = current; app->current_output = NULL; @@ -278,7 +278,7 @@ get_current_modes (App *app) { MateRROutput *output; - if (app->current_configuration->clone) + if (mate_rr_config_get_clone (app->current_configuration)) { return mate_rr_screen_list_clone_modes (app->screen); } @@ -287,8 +287,8 @@ get_current_modes (App *app) if (!app->current_output) return NULL; - output = mate_rr_screen_get_output_by_name ( - app->screen, app->current_output->name); + output = mate_rr_screen_get_output_by_name (app->screen, + mate_rr_output_info_get_name (app->current_output)); if (!output) return NULL; @@ -317,20 +317,20 @@ rebuild_rotation_combo (App *app) clear_combo (app->rotation_combo); - gtk_widget_set_sensitive ( - app->rotation_combo, app->current_output && app->current_output->on); + gtk_widget_set_sensitive (app->rotation_combo, + app->current_output && mate_rr_output_info_get_active (app->current_output)); if (!app->current_output) return; - current = app->current_output->rotation; + current = mate_rr_output_info_get_rotation (app->current_output); selection = NULL; for (i = 0; i < G_N_ELEMENTS (rotations); ++i) { const RotationInfo *info = &(rotations[i]); - app->current_output->rotation = info->rotation; + mate_rr_output_info_set_rotation (app->current_output, info->rotation); /* NULL-GError --- FIXME: we should say why this rotation is not available! */ if (mate_rr_config_applicable (app->current_configuration, app->screen, NULL)) @@ -342,7 +342,7 @@ rebuild_rotation_combo (App *app) } } - app->current_output->rotation = current; + mate_rr_output_info_set_rotation (app->current_output, current); if (!(selection && combo_select (app->rotation_combo, selection))) combo_select (app->rotation_combo, _("Normal")); @@ -365,7 +365,7 @@ rebuild_rate_combo (App *app) clear_combo (app->refresh_combo); gtk_widget_set_sensitive ( - app->refresh_combo, app->current_output && app->current_output->on); + app->refresh_combo, app->current_output && mate_rr_output_info_get_active (app->current_output)); if (!app->current_output || !(modes = get_current_modes (app))) @@ -379,13 +379,16 @@ rebuild_rate_combo (App *app) { MateRRMode *mode = modes[i]; int width, height, rate; + int output_width, output_height; + + mate_rr_output_info_get_geometry (app->current_output, NULL, NULL, &output_width, &output_height); width = mate_rr_mode_get_width (mode); height = mate_rr_mode_get_height (mode); rate = mate_rr_mode_get_freq (mode); - if (width == app->current_output->width && - height == app->current_output->height) + if (width == output_width && + height == output_height) { add_key (app->refresh_combo, idle_free (make_rate_string (rate)), @@ -396,7 +399,7 @@ rebuild_rate_combo (App *app) } } - if (!combo_select (app->refresh_combo, idle_free (make_rate_string (app->current_output->rate)))) + if (!combo_select (app->refresh_combo, idle_free (make_rate_string (mate_rr_output_info_get_refresh_rate (app->current_output))))) combo_select (app->refresh_combo, idle_free (make_rate_string (best))); } @@ -404,11 +407,11 @@ static int count_active_outputs (App *app) { int i, count = 0; + MateRROutputInfo **outputs = mate_rr_config_get_outputs (app->current_configuration); - for (i = 0; app->current_configuration->outputs[i] != NULL; ++i) + for (i = 0; outputs[i] != NULL; ++i) { - MateOutputInfo *output = app->current_configuration->outputs[i]; - if (output->on) + if (mate_rr_output_info_get_active (outputs[i])) count++; } @@ -420,8 +423,9 @@ static int count_all_outputs (MateRRConfig *config) { int i; + MateRROutputInfo **outputs = mate_rr_config_get_outputs (config); - for (i = 0; config->outputs[i] != NULL; i++) + for (i = 0; outputs[i] != NULL; i++) ; return i; @@ -448,18 +452,17 @@ mirror_screens_is_supported (App *app) if (have_clone_size) { int i; int num_outputs_with_clone_size; + MateRROutputInfo **outputs = mate_rr_config_get_outputs (app->current_configuration); num_outputs_with_clone_size = 0; - for (i = 0; app->current_configuration->outputs[i] != NULL; i++) + for (i = 0; outputs[i] != NULL; i++) { - MateOutputInfo *output = app->current_configuration->outputs[i]; - /* We count the connected outputs that support the clone size. It * doesn't matter if those outputs aren't actually On currently; we * will turn them on in on_clone_changed(). */ - if (output->connected && output_info_supports_mode (app, output, clone_width, clone_height)) + if (mate_rr_output_info_get_connected (outputs[i]) && output_info_supports_mode (app, outputs[i], clone_width, clone_height)) num_outputs_with_clone_size++; } @@ -478,7 +481,7 @@ rebuild_mirror_screens (App *app) g_signal_handlers_block_by_func (app->clone_checkbox, G_CALLBACK (on_clone_changed), app); - mirror_is_active = app->current_configuration && app->current_configuration->clone; + mirror_is_active = app->current_configuration && mate_rr_config_get_clone (app->current_configuration); /* If mirror_is_active, then it *must* be possible to turn mirroring off */ mirror_is_supported = mirror_is_active || mirror_screens_is_supported (app); @@ -502,10 +505,10 @@ rebuild_current_monitor_label (App *app) if (app->current_output) { - if (app->current_configuration->clone) + if (mate_rr_config_get_clone (app->current_configuration)) tmp = g_strdup (_("Mirror Screens")); else - tmp = g_strdup_printf (_("Monitor: %s"), app->current_output->display_name); + tmp = g_strdup_printf (_("Monitor: %s"), mate_rr_output_info_get_display_name (app->current_output)); str = g_strdup_printf ("%s", tmp); #if GTK_CHECK_VERSION (3, 0, 0) @@ -577,14 +580,14 @@ rebuild_on_off_radios (App *app) on_active = FALSE; off_active = FALSE; - if (!app->current_configuration->clone && app->current_output) + if (!mate_rr_config_get_clone (app->current_configuration) && app->current_output) { - if (count_active_outputs (app) > 1 || !app->current_output->on) + if (count_active_outputs (app) > 1 || !mate_rr_output_info_get_active (app->current_output)) sensitive = TRUE; else sensitive = FALSE; - on_active = app->current_output->on; + on_active = mate_rr_output_info_get_active (app->current_output); off_active = !on_active; } @@ -633,19 +636,22 @@ rebuild_resolution_combo (App *app) int i; MateRRMode **modes; const char *current; + int output_width, output_height; clear_combo (app->resolution_combo); if (!(modes = get_current_modes (app)) || !app->current_output - || !app->current_output->on) + || !mate_rr_output_info_get_active (app->current_output)) { gtk_widget_set_sensitive (app->resolution_combo, FALSE); return; } g_assert (app->current_output != NULL); - g_assert (app->current_output->width != 0 && app->current_output->height != 0); + + mate_rr_output_info_get_geometry (app->current_output, NULL, NULL, &output_width, &output_height); + g_assert (output_width != 0 && output_height != 0); gtk_widget_set_sensitive (app->resolution_combo, TRUE); @@ -661,7 +667,7 @@ rebuild_resolution_combo (App *app) width, height, 0, -1); } - current = idle_free (make_resolution_string (app->current_output->width, app->current_output->height)); + current = idle_free (make_resolution_string (output_width, output_height)); if (!combo_select (app->resolution_combo, current)) { @@ -687,7 +693,7 @@ rebuild_gui (App *app) sensitive = app->current_output ? TRUE : FALSE; #if 0 - g_debug ("rebuild gui, is on: %d", app->current_output->on); + g_debug ("rebuild gui, is on: %d", mate_rr_output_info_get_active (app->current_output)); #endif rebuild_mirror_screens (app); @@ -698,7 +704,7 @@ rebuild_gui (App *app) rebuild_rotation_combo (app); #if 0 - g_debug ("sensitive: %d, on: %d", sensitive, app->current_output->on); + g_debug ("sensitive: %d, on: %d", sensitive, mate_rr_output_info_get_active (app->current_output)); #endif gtk_widget_set_sensitive (app->panel_checkbox, sensitive); @@ -750,7 +756,7 @@ on_rotation_changed (GtkComboBox *box, gpointer data) return; if (get_mode (app->rotation_combo, NULL, NULL, NULL, &rotation)) - app->current_output->rotation = rotation; + mate_rr_output_info_set_rotation (app->current_output, rotation); foo_scroll_area_invalidate (FOO_SCROLL_AREA (app->area)); } @@ -765,7 +771,7 @@ on_rate_changed (GtkComboBox *box, gpointer data) return; if (get_mode (app->refresh_combo, NULL, NULL, &rate, NULL)) - app->current_output->rate = rate; + rate = mate_rr_output_info_get_refresh_rate (app->current_output); foo_scroll_area_invalidate (FOO_SCROLL_AREA (app->area)); } @@ -775,11 +781,15 @@ select_resolution_for_current_output (App *app) { MateRRMode **modes; int width, height; + int x, y; + mate_rr_output_info_get_geometry (app->current_output, &x, &y, NULL, NULL); + + width = mate_rr_output_info_get_preferred_width (app->current_output); + height = mate_rr_output_info_get_preferred_height (app->current_output); - if (app->current_output->pref_width != 0 && app->current_output->pref_height != 0) + if (width != 0 && height != 0) { - app->current_output->width = app->current_output->pref_width; - app->current_output->height = app->current_output->pref_height; + mate_rr_output_info_set_geometry (app->current_output, x, y, width, height); return; } @@ -789,8 +799,7 @@ select_resolution_for_current_output (App *app) find_best_mode (modes, &width, &height); - app->current_output->width = width; - app->current_output->height = height; + mate_rr_output_info_set_geometry (app->current_output, x, y, width, height); } static void @@ -815,7 +824,7 @@ monitor_on_off_toggled_cb (GtkToggleButton *toggle, gpointer data) return; } - app->current_output->on = is_on; + mate_rr_output_info_set_active (app->current_output, is_on); if (is_on) select_resolution_for_current_output (app); /* The refresh rate will be picked in rebuild_rate_combo() */ @@ -825,7 +834,7 @@ monitor_on_off_toggled_cb (GtkToggleButton *toggle, gpointer data) } static void -realign_outputs_after_resolution_change (App *app, MateOutputInfo *output_that_changed, int old_width, int old_height) +realign_outputs_after_resolution_change (App *app, MateRROutputInfo *output_that_changed, int old_width, int old_height) { /* We find the outputs that were below or to the right of the output that * changed, and realign them; we also do that for outputs that shared the @@ -836,38 +845,45 @@ realign_outputs_after_resolution_change (App *app, MateOutputInfo *output_that_c int i; int old_right_edge, old_bottom_edge; int dx, dy; + int x, y, width, height; + MateRROutputInfo **outputs; g_assert (app->current_configuration != NULL); - if (output_that_changed->width == old_width && output_that_changed->height == old_height) + mate_rr_output_info_get_geometry (output_that_changed, &x, &y, &width, &height); + if (width == old_width && height == old_height) return; - old_right_edge = output_that_changed->x + old_width; - old_bottom_edge = output_that_changed->y + old_height; + old_right_edge = x + old_width; + old_bottom_edge = y + old_height; - dx = output_that_changed->width - old_width; - dy = output_that_changed->height - old_height; + dx = width - old_width; + dy = height - old_height; - for (i = 0; app->current_configuration->outputs[i] != NULL; i++) { - MateOutputInfo *output; + outputs = mate_rr_config_get_outputs (app->current_configuration); + + for (i = 0; outputs[i] != NULL; i++) + { + int output_x, output_y; int output_width, output_height; - output = app->current_configuration->outputs[i]; + if (outputs[i] == output_that_changed || mate_rr_output_info_get_connected (outputs[i])) + continue; - if (output == output_that_changed || !output->connected) - continue; + mate_rr_output_info_get_geometry (outputs[i], &output_x, &output_y, &output_width, &output_height); - get_geometry (output, &output_width, &output_height); + if (output_x >= old_right_edge) + output_x += dx; + else if (output_x + output_width == old_right_edge) + output_x = x + width - output_width; - if (output->x >= old_right_edge) - output->x += dx; - else if (output->x + output_width == old_right_edge) - output->x = output_that_changed->x + output_that_changed->width - output_width; - if (output->y >= old_bottom_edge) - output->y += dy; - else if (output->y + output_height == old_bottom_edge) - output->y = output_that_changed->y + output_that_changed->height - output_height; + if (output_y >= old_bottom_edge) + output_y += dy; + else if (output_y + output_height == old_bottom_edge) + output_y = y + height - output_height; + + mate_rr_output_info_set_geometry (outputs[i], output_x, output_y, output_width, output_height); } } @@ -876,24 +892,23 @@ on_resolution_changed (GtkComboBox *box, gpointer data) { App *app = data; int old_width, old_height; + int x, y; int width; int height; if (!app->current_output) return; - old_width = app->current_output->width; - old_height = app->current_output->height; + mate_rr_output_info_get_geometry (app->current_output, &x, &y, &old_width, &old_height); if (get_mode (app->resolution_combo, &width, &height, NULL, NULL)) { - app->current_output->width = width; - app->current_output->height = height; + mate_rr_output_info_set_geometry (app->current_output, x, y, width, height); if (width == 0 || height == 0) - app->current_output->on = FALSE; + mate_rr_output_info_set_active (app->current_output, FALSE); else - app->current_output->on = TRUE; + mate_rr_output_info_set_active (app->current_output, TRUE); } realign_outputs_after_resolution_change (app, app->current_output, old_width, old_height); @@ -909,6 +924,7 @@ lay_out_outputs_horizontally (App *app) { int i; int x; + MateRROutputInfo **outputs; /* Lay out all the monitors horizontally when "mirror screens" is turned * off, to avoid having all of them overlapped initially. We put the @@ -918,30 +934,29 @@ lay_out_outputs_horizontally (App *app) x = 0; /* First pass, all "on" outputs */ + outputs = mate_rr_config_get_outputs (app->current_configuration); - for (i = 0; app->current_configuration->outputs[i]; ++i) + for (i = 0; outputs[i]; ++i) { - MateOutputInfo *output; - - output = app->current_configuration->outputs[i]; - if (output->connected && output->on) { - output->x = x; - output->y = 0; - x += output->width; + int width, height; + if (mate_rr_output_info_get_connected (outputs[i]) &&mate_rr_output_info_get_active (outputs[i])) + { + mate_rr_output_info_get_geometry (outputs[i], NULL, NULL, &width, &height); + mate_rr_output_info_set_geometry (outputs[i], x, 0, width, height); + x += width; } } /* Second pass, all the black screens */ - for (i = 0; app->current_configuration->outputs[i]; ++i) + for (i = 0; outputs[i]; ++i) { - MateOutputInfo *output; - - output = app->current_configuration->outputs[i]; - if (!(output->connected && output->on)) { - output->x = x; - output->y = 0; - x += output->width; + int width, height; + if (!(mate_rr_output_info_get_connected (outputs[i]) && mate_rr_output_info_get_active (outputs[i]))) + { + mate_rr_output_info_get_geometry (outputs[i], NULL, NULL, &width, &height); + mate_rr_output_info_set_geometry (outputs[i], x, 0, width, height); + x += width; } } @@ -986,16 +1001,16 @@ get_clone_size (MateRRScreen *screen, int *width, int *height) } static gboolean -output_info_supports_mode (App *app, MateOutputInfo *info, int width, int height) +output_info_supports_mode (App *app, MateRROutputInfo *info, int width, int height) { MateRROutput *output; MateRRMode **modes; int i; - if (!info->connected) + if (!mate_rr_output_info_get_connected (info)) return FALSE; - output = mate_rr_screen_get_output_by_name (app->screen, info->name); + output = mate_rr_screen_get_output_by_name (app->screen, mate_rr_output_info_get_name (info)); if (!output) return FALSE; @@ -1015,19 +1030,19 @@ on_clone_changed (GtkWidget *box, gpointer data) { App *app = data; - app->current_configuration->clone = - gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (app->clone_checkbox)); + mate_rr_config_set_clone (app->current_configuration, gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (app->clone_checkbox))); - if (app->current_configuration->clone) + if (mate_rr_config_get_clone (app->current_configuration)) { int i; int width, height; + MateRROutputInfo **outputs = mate_rr_config_get_outputs (app->current_configuration); - for (i = 0; app->current_configuration->outputs[i]; ++i) + for (i = 0; outputs[i]; ++i) { - if (app->current_configuration->outputs[i]->connected) + if (mate_rr_output_info_get_connected(outputs[i])) { - app->current_output = app->current_configuration->outputs[i]; + app->current_output = outputs[i]; break; } } @@ -1039,11 +1054,12 @@ on_clone_changed (GtkWidget *box, gpointer data) get_clone_size (app->screen, &width, &height); - for (i = 0; app->current_configuration->outputs[i]; i++) { - if (output_info_supports_mode (app, app->current_configuration->outputs[i], width, height)) { - app->current_configuration->outputs[i]->on = TRUE; - app->current_configuration->outputs[i]->width = width; - app->current_configuration->outputs[i]->height = height; + for (i = 0; outputs[i]; i++) { + int x, y; + if (output_info_supports_mode (app, outputs[i], width, height)) { + mate_rr_output_info_set_active (outputs[i], TRUE); + mate_rr_output_info_get_geometry (outputs[i], &x, &y, NULL, NULL); + mate_rr_output_info_set_geometry (outputs[i], x, y, width, height); } } } @@ -1057,19 +1073,21 @@ on_clone_changed (GtkWidget *box, gpointer data) } static void -get_geometry (MateOutputInfo *output, int *w, int *h) +get_geometry (MateRROutputInfo *output, int *w, int *h) { - if (output->on) + MateRRRotation rotation; + + if (mate_rr_output_info_get_active (output)) { - *h = output->height; - *w = output->width; + mate_rr_output_info_get_geometry (output, NULL, NULL, w, h); } else { - *h = output->pref_height; - *w = output->pref_width; + *h = mate_rr_output_info_get_preferred_height (output); + *w = mate_rr_output_info_get_preferred_width (output); } - if ((output->rotation & MATE_RR_ROTATION_90) || (output->rotation & MATE_RR_ROTATION_270)) + rotation = mate_rr_output_info_get_rotation (output); + if ((rotation & MATE_RR_ROTATION_90) || (rotation & MATE_RR_ROTATION_270)) { int tmp; tmp = *h; @@ -1086,6 +1104,7 @@ list_connected_outputs (App *app, int *total_w, int *total_h) { int i, dummy; GList *result = NULL; + MateRROutputInfo **outputs; if (!total_w) total_w = &dummy; @@ -1094,17 +1113,17 @@ list_connected_outputs (App *app, int *total_w, int *total_h) *total_w = 0; *total_h = 0; - for (i = 0; app->current_configuration->outputs[i] != NULL; ++i) - { - MateOutputInfo *output = app->current_configuration->outputs[i]; - if (output->connected) + outputs = mate_rr_config_get_outputs(app->current_configuration); + for (i = 0; outputs[i] != NULL; ++i) + { + if (mate_rr_output_info_get_connected (outputs[i])) { int w, h; - result = g_list_prepend (result, output); + result = g_list_prepend (result, outputs[i]); - get_geometry (output, &w, &h); + get_geometry (outputs[i], &w, &h); *total_w += w; *total_h += h; @@ -1150,7 +1169,7 @@ compute_scale (App *app) typedef struct Edge { - MateOutputInfo *output; + MateRROutputInfo *output; int x1, y1; int x2, y2; } Edge; @@ -1163,7 +1182,7 @@ typedef struct Snap } Snap; static void -add_edge (MateOutputInfo *output, int x1, int y1, int x2, int y2, GArray *edges) +add_edge (MateRROutputInfo *output, int x1, int y1, int x2, int y2, GArray *edges) { Edge e; @@ -1177,13 +1196,11 @@ add_edge (MateOutputInfo *output, int x1, int y1, int x2, int y2, GArray *edges) } static void -list_edges_for_output (MateOutputInfo *output, GArray *edges) +list_edges_for_output (MateRROutputInfo *output, GArray *edges) { int x, y, w, h; - x = output->x; - y = output->y; - get_geometry (output, &w, &h); + mate_rr_output_info_get_geometry (output, &x, &y, &w, &h); /* Top, Bottom, Left, Right */ add_edge (output, x, y, x + w, y, edges); @@ -1196,13 +1213,12 @@ static void list_edges (MateRRConfig *config, GArray *edges) { int i; + MateRROutputInfo **outputs = mate_rr_config_get_outputs (config); - for (i = 0; config->outputs[i]; ++i) + for (i = 0; outputs[i]; ++i) { - MateOutputInfo *output = config->outputs[i]; - - if (output->connected) - list_edges_for_output (output, edges); + if (mate_rr_output_info_get_connected (outputs[i])) + list_edges_for_output (outputs[i], edges); } } @@ -1287,7 +1303,7 @@ add_edge_snaps (Edge *snapper, Edge *snappee, GArray *snaps) } static void -list_snaps (MateOutputInfo *output, GArray *edges, GArray *snaps) +list_snaps (MateRROutputInfo *output, GArray *edges, GArray *snaps) { int i; @@ -1343,7 +1359,7 @@ edges_align (Edge *e1, Edge *e2) } static gboolean -output_is_aligned (MateOutputInfo *output, GArray *edges) +output_is_aligned (MateRROutputInfo *output, GArray *edges) { gboolean result = FALSE; int i; @@ -1380,35 +1396,28 @@ done: } static void -get_output_rect (MateOutputInfo *output, GdkRectangle *rect) +get_output_rect (MateRROutputInfo *output, GdkRectangle *rect) { - int w, h; - - get_geometry (output, &w, &h); - - rect->width = w; - rect->height = h; - rect->x = output->x; - rect->y = output->y; + mate_rr_output_info_get_geometry (output, &rect->x, &rect->y, &rect->width, &rect->height); } static gboolean -output_overlaps (MateOutputInfo *output, MateRRConfig *config) +output_overlaps (MateRROutputInfo *output, MateRRConfig *config) { int i; GdkRectangle output_rect; + MateRROutputInfo **outputs; get_output_rect (output, &output_rect); - for (i = 0; config->outputs[i]; ++i) + outputs = mate_rr_config_get_outputs (config); + for (i = 0; outputs[i]; ++i) { - MateOutputInfo *other = config->outputs[i]; - - if (other != output && other->connected) + if (outputs[i] != output && mate_rr_output_info_get_connected (outputs[i])) { GdkRectangle other_rect; - get_output_rect (other, &other_rect); + get_output_rect (outputs[i], &other_rect); if (gdk_rectangle_intersect (&output_rect, &other_rect, NULL)) return TRUE; } @@ -1422,17 +1431,17 @@ mate_rr_config_is_aligned (MateRRConfig *config, GArray *edges) { int i; gboolean result = TRUE; + MateRROutputInfo **outputs; - for (i = 0; config->outputs[i]; ++i) + outputs = mate_rr_config_get_outputs(config); + for (i = 0; outputs[i]; ++i) { - MateOutputInfo *output = config->outputs[i]; - - if (output->connected) + if (mate_rr_output_info_get_connected (outputs[i])) { - if (!output_is_aligned (output, edges)) + if (!output_is_aligned (outputs[i], edges)) return FALSE; - if (output_overlaps (output, config)) + if (output_overlaps (outputs[i], config)) return FALSE; } } @@ -1537,14 +1546,14 @@ on_output_event (FooScrollArea *area, FooScrollAreaEvent *event, gpointer data) { - MateOutputInfo *output = data; + MateRROutputInfo *output = data; App *app = g_object_get_data (G_OBJECT (area), "app"); /* If the mouse is inside the outputs, set the cursor to "you can move me". See * on_canvas_event() for where we reset the cursor to the default if it * exits the outputs' area. */ - if (!app->current_configuration->clone && get_n_connected (app) > 1) + if (!mate_rr_config_get_clone (app->current_configuration) && get_n_connected (app) > 1) set_cursor (GTK_WIDGET (area), GDK_FLEUR); if (event->type == FOO_BUTTON_PRESS) @@ -1556,17 +1565,20 @@ on_output_event (FooScrollArea *area, rebuild_gui (app); set_monitors_tooltip (app, TRUE); - if (!app->current_configuration->clone && get_n_connected (app) > 1) + if (!mate_rr_config_get_clone (app->current_configuration) && get_n_connected (app) > 1) { + int output_x, output_y; + mate_rr_output_info_get_geometry (output, &output_x, &output_y, NULL, NULL); + foo_scroll_area_begin_grab (area, on_output_event, data); info = g_new0 (GrabInfo, 1); info->grab_x = event->x; info->grab_y = event->y; - info->output_x = output->x; - info->output_y = output->y; + info->output_x = output_x; + info->output_y = output_y; - output->user_data = info; + g_object_set_data (G_OBJECT (output), "grab-info", info); } foo_scroll_area_invalidate (area); @@ -1575,20 +1587,19 @@ on_output_event (FooScrollArea *area, { if (foo_scroll_area_is_grabbed (area)) { - GrabInfo *info = output->user_data; + GrabInfo *info = g_object_get_data (G_OBJECT (output), "grab-info"); double scale = compute_scale (app); int old_x, old_y; + int width, height; int new_x, new_y; int i; GArray *edges, *snaps, *new_edges; - old_x = output->x; - old_y = output->y; + mate_rr_output_info_get_geometry (output, &old_x, &old_y, &width, &height); new_x = info->output_x + (event->x - info->grab_x) / scale; new_y = info->output_y + (event->y - info->grab_y) / scale; - output->x = new_x; - output->y = new_y; + mate_rr_output_info_set_geometry (output, new_x, new_y, width, height); edges = g_array_new (TRUE, TRUE, sizeof (Edge)); snaps = g_array_new (TRUE, TRUE, sizeof (Snap)); @@ -1599,16 +1610,14 @@ on_output_event (FooScrollArea *area, g_array_sort (snaps, compare_snaps); - output->x = info->output_x; - output->y = info->output_y; + mate_rr_output_info_set_geometry (output, new_x, new_y, width, height); for (i = 0; i < snaps->len; ++i) { Snap *snap = &(g_array_index (snaps, Snap, i)); GArray *new_edges = g_array_new (TRUE, TRUE, sizeof (Edge)); - output->x = new_x + snap->dx; - output->y = new_y + snap->dy; + mate_rr_output_info_set_geometry (output, new_x + snap->dx, new_y + snap->dy, width, height); g_array_set_size (new_edges, 0); list_edges (app->current_configuration, new_edges); @@ -1620,8 +1629,7 @@ on_output_event (FooScrollArea *area, } else { - output->x = info->output_x; - output->y = info->output_y; + mate_rr_output_info_set_geometry (output, info->output_x, info->output_y, width, height); } } @@ -1634,8 +1642,8 @@ on_output_event (FooScrollArea *area, foo_scroll_area_end_grab (area); set_monitors_tooltip (app, FALSE); - g_free (output->user_data); - output->user_data = NULL; + g_free (g_object_get_data (G_OBJECT (output), "grab-info")); + g_object_set_data (G_OBJECT (output), "grab-info", NULL); #if 0 g_debug ("new position: %d %d %d %d", output->x, output->y, output->width, output->height); @@ -1661,11 +1669,11 @@ on_canvas_event (FooScrollArea *area, static PangoLayout * get_display_name (App *app, - MateOutputInfo *output) + MateRROutputInfo *output) { const char *text; - if (app->current_configuration->clone) { + if (mate_rr_config_get_clone (app->current_configuration)) { /* Translators: this is the feature where what you see on your laptop's * screen is the same as your external monitor. Here, "Mirror" is being * used as an adjective, not as a verb. For example, the Spanish @@ -1673,7 +1681,7 @@ get_display_name (App *app, */ text = _("Mirror Screens"); } else - text = output->display_name; + text = mate_rr_output_info_get_display_name (output); return gtk_widget_create_pango_layout ( GTK_WIDGET (app->area), text); @@ -1743,9 +1751,11 @@ paint_output (App *app, cairo_t *cr, int i) int w, h; double scale = compute_scale (app); double x, y; + int output_x, output_y; + MateRRRotation rotation; int total_w, total_h; GList *connected_outputs = list_connected_outputs (app, &total_w, &total_h); - MateOutputInfo *output = g_list_nth (connected_outputs, i)->data; + MateRROutputInfo *output = g_list_nth (connected_outputs, i)->data; PangoLayout *layout = get_display_name (app, output); PangoRectangle ink_extent, log_extent; GdkRectangle viewport; @@ -1772,8 +1782,9 @@ paint_output (App *app, cairo_t *cr, int i) viewport.height -= 2 * MARGIN; viewport.width -= 2 * MARGIN; - x = output->x * scale + MARGIN + (viewport.width - total_w * scale) / 2.0; - y = output->y * scale + MARGIN + (viewport.height - total_h * scale) / 2.0; + mate_rr_output_info_get_geometry (output, &output_x, &output_y, NULL, NULL); + x = output_x * scale + MARGIN + (viewport.width - total_w * scale) / 2.0; + y = output_y * scale + MARGIN + (viewport.height - total_h * scale) / 2.0; #if 0 g_debug ("scaled: %f %f", x, y); @@ -1789,10 +1800,11 @@ paint_output (App *app, cairo_t *cr, int i) /* rotation is already applied in get_geometry */ - if (output->rotation & MATE_RR_REFLECT_X) + rotation = mate_rr_output_info_get_rotation (output); + if (rotation & MATE_RR_REFLECT_X) cairo_scale (cr, -1, 1); - if (output->rotation & MATE_RR_REFLECT_Y) + if (rotation & MATE_RR_REFLECT_Y) cairo_scale (cr, 1, -1); cairo_translate (cr, @@ -1815,7 +1827,7 @@ paint_output (App *app, cairo_t *cr, int i) b = output_color.blue / 65535.0; #endif - if (!output->on) + if (!mate_rr_output_info_get_active (output)) { /* If the output is turned off, just darken the selected color */ r *= 0.2; @@ -1861,7 +1873,7 @@ paint_output (App *app, cairo_t *cr, int i) cairo_scale (cr, factor, factor); - if (output->on) + if (mate_rr_output_info_get_active (output)) cairo_set_source_rgb (cr, 0.0, 0.0, 0.0); else cairo_set_source_rgb (cr, 1.0, 1.0, 1.0); @@ -1908,7 +1920,7 @@ on_area_paint (FooScrollArea *area, { paint_output (app, cr, g_list_position (connected_outputs, list)); - if (app->current_configuration->clone) + if (mate_rr_config_get_clone (app->current_configuration)) break; } } @@ -1951,19 +1963,19 @@ compute_virtual_size_for_configuration (MateRRConfig *config, int *ret_width, in { int i; int width, height; + int output_x, output_y, output_width, output_height; + MateRROutputInfo **outputs; width = height = 0; - for (i = 0; config->outputs[i] != NULL; i++) + outputs = mate_rr_config_get_outputs (config); + for (i = 0; outputs[i] != NULL; i++) { - MateOutputInfo *output; - - output = config->outputs[i]; - - if (output->on) + if (mate_rr_output_info_get_active (outputs[i])) { - width = MAX (width, output->x + output->width); - height = MAX (height, output->y + output->height); + mate_rr_output_info_get_geometry (outputs[i], &output_x, &output_y, &output_width, &output_height); + width = MAX (width, output_x + output_width); + height = MAX (height, output_y + output_height); } } @@ -2066,11 +2078,11 @@ ensure_current_configuration_is_saved (void) if (!rr_screen) return; - rr_config = mate_rr_config_new_current (rr_screen); + rr_config = mate_rr_config_new_current (rr_screen, NULL); mate_rr_config_save (rr_config, NULL); /* NULL-GError */ - mate_rr_config_free (rr_config); - mate_rr_screen_destroy (rr_screen); + g_object_unref (rr_config); + g_object_unref (rr_screen); } /* Callback for dbus_g_proxy_begin_call() */ @@ -2183,7 +2195,7 @@ driver_is_randr_10 (MateRRConfig *config) * or is mate_rr_screen_new()'s return value sufficient? */ - return (count_all_outputs (config) == 1 && strcmp (config->outputs[0]->name, "default") == 0); + return (count_all_outputs (config) == 1 && strcmp (mate_rr_output_info_get_name (mate_rr_config_get_outputs (config)[0]), "default") == 0); } #endif @@ -2216,37 +2228,38 @@ on_show_icon_toggled (GtkWidget *widget, gpointer data) gtk_toggle_button_get_active (tb)); } -static MateOutputInfo * +static MateRROutputInfo * get_nearest_output (MateRRConfig *configuration, int x, int y) { int i; int nearest_index; int nearest_dist; + MateRROutputInfo **outputs; nearest_index = -1; nearest_dist = G_MAXINT; - for (i = 0; configuration->outputs[i] != NULL; i++) + outputs = mate_rr_config_get_outputs (configuration); + for (i = 0; outputs[i] != NULL; i++) { - MateOutputInfo *output; int dist_x, dist_y; + int output_x, output_y, output_width, output_height; - output = configuration->outputs[i]; - - if (!(output->connected && output->on)) + if (!(mate_rr_output_info_get_connected(outputs[i]) && mate_rr_output_info_get_active (outputs[i]))) continue; - if (x < output->x) - dist_x = output->x - x; - else if (x >= output->x + output->width) - dist_x = x - (output->x + output->width) + 1; + mate_rr_output_info_get_geometry (outputs[i], &output_x, &output_y, &output_width, &output_height); + if (x < output_x) + dist_x = output_x - x; + else if (x >= output_x + output_width) + dist_x = x - (output_x + output_width) + 1; else dist_x = 0; - if (y < output->y) - dist_y = output->y - y; - else if (y >= output->y + output->height) - dist_y = y - (output->y + output->height) + 1; + if (y < output_y) + dist_y = output_y - y; + else if (y >= output_y + output_height) + dist_y = y - (output_y + output_height) + 1; else dist_y = 0; @@ -2258,7 +2271,7 @@ get_nearest_output (MateRRConfig *configuration, int x, int y) } if (nearest_index != -1) - return configuration->outputs[nearest_index]; + return outputs[nearest_index]; else return NULL; @@ -2267,13 +2280,14 @@ get_nearest_output (MateRRConfig *configuration, int x, int y) /* Gets the output that contains the largest intersection with the window. * Logic stolen from gdk_screen_get_monitor_at_window(). */ -static MateOutputInfo * +static MateRROutputInfo * get_output_for_window (MateRRConfig *configuration, GdkWindow *window) { GdkRectangle win_rect; int i; int largest_area; int largest_index; + MateRROutputInfo **outputs; #if GTK_CHECK_VERSION (3, 0, 0) gdk_window_get_geometry (window, &win_rect.x, &win_rect.y, &win_rect.width, &win_rect.height); @@ -2285,19 +2299,14 @@ get_output_for_window (MateRRConfig *configuration, GdkWindow *window) largest_area = 0; largest_index = -1; - for (i = 0; configuration->outputs[i] != NULL; i++) + outputs = mate_rr_config_get_outputs (configuration); + for (i = 0; outputs[i] != NULL; i++) { - MateOutputInfo *output; GdkRectangle output_rect, intersection; - output = configuration->outputs[i]; - - output_rect.x = output->x; - output_rect.y = output->y; - output_rect.width = output->width; - output_rect.height = output->height; + mate_rr_output_info_get_geometry (outputs[i], &output_rect.x, &output_rect.y, &output_rect.width, &output_rect.height); - if (output->connected && gdk_rectangle_intersect (&win_rect, &output_rect, &intersection)) + if (mate_rr_output_info_get_connected (outputs[i]) && gdk_rectangle_intersect (&win_rect, &output_rect, &intersection)) { int area; @@ -2311,7 +2320,7 @@ get_output_for_window (MateRRConfig *configuration, GdkWindow *window) } if (largest_index != -1) - return configuration->outputs[largest_index]; + return outputs[largest_index]; else return get_nearest_output (configuration, win_rect.x + win_rect.width / 2, @@ -2615,7 +2624,7 @@ restart: } gtk_widget_destroy (app->dialog); - mate_rr_screen_destroy (app->screen); + g_object_unref (app->screen); g_object_unref (app->settings); } -- cgit v1.2.1