summaryrefslogtreecommitdiff
path: root/capplets/display
diff options
context:
space:
mode:
authorinfirit <[email protected]>2014-07-09 00:52:37 +0200
committerinfirit <[email protected]>2014-11-30 18:28:23 +0100
commite9bf4efa713959e68d61db7fbca93f7dfea2ae5e (patch)
tree1db42eab58bf7c58fddf9d9b6ef28c24c158f2de /capplets/display
parent94eaf6caec20abd9b3265e99aab722e8f4851972 (diff)
downloadmate-control-center-e9bf4efa713959e68d61db7fbca93f7dfea2ae5e.tar.bz2
mate-control-center-e9bf4efa713959e68d61db7fbca93f7dfea2ae5e.tar.xz
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 <[email protected]>
Diffstat (limited to 'capplets/display')
-rw-r--r--capplets/display/xrandr-capplet.c443
1 files changed, 226 insertions, 217 deletions
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 ("<b>%s</b>", 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);
}