From fd55cee48a6fa2fd5ee312923abe2c2893f519f8 Mon Sep 17 00:00:00 2001 From: infirit Date: Sat, 8 Nov 2014 00:59:16 +0100 Subject: Turn GnomeRRConfig and GnomeOutputInfo into GObjects Based on gnome-desktop commits: 1d1f8ef64f404eccb29d384a75939701327e8e77 2c7a2e90fee98ff61e530098bc962a65dec670bc 7436f759bec00d87f597ef0b6945da8fa8969e85 From: Giovanni Campagna Gnome Bug: https://bugzilla.gnome.org/show_bug.cgi?id=630913 --- libmate-desktop/mate-rr-config.c | 837 +++++++++++++++++++-------------------- 1 file changed, 414 insertions(+), 423 deletions(-) (limited to 'libmate-desktop/mate-rr-config.c') diff --git a/libmate-desktop/mate-rr-config.c b/libmate-desktop/mate-rr-config.c index 9f83c0c..1c13630 100644 --- a/libmate-desktop/mate-rr-config.c +++ b/libmate-desktop/mate-rr-config.c @@ -1,6 +1,8 @@ /* mate-rr-config.c + * -*- c-basic-offset: 4 -*- * * Copyright 2007, 2008, Red Hat, Inc. + * Copyright 2010 Giovanni Campagna * * This file is part of the Mate Library. * @@ -34,7 +36,6 @@ #include #include -#undef MATE_DISABLE_DEPRECATED #include "mate-rr-config.h" #include "edid.h" @@ -79,16 +80,18 @@ typedef struct CrtcAssignment CrtcAssignment; static gboolean crtc_assignment_apply (CrtcAssignment *assign, guint32 timestamp, GError **error); -static CrtcAssignment *crtc_assignment_new (MateRRScreen *screen, - MateOutputInfo **outputs, - GError **error); +static CrtcAssignment *crtc_assignment_new (MateRRScreen *screen, + MateRROutputInfo **outputs, + GError **error); static void crtc_assignment_free (CrtcAssignment *assign); -static void output_free (MateOutputInfo *output); -static MateOutputInfo *output_copy (const MateOutputInfo *output); -static MateRRConfig * mate_rr_config_copy (const MateRRConfig *config); -G_DEFINE_BOXED_TYPE (MateOutputInfo, mate_rr_output_info, output_copy, output_free) -G_DEFINE_BOXED_TYPE (MateRRConfig, mate_rr_config, mate_rr_config_copy, mate_rr_config_free) +enum { + PROP_0, + PROP_SCREEN, + PROP_LAST +}; + +G_DEFINE_TYPE (MateRRConfig, mate_rr_config, G_TYPE_OBJECT) typedef struct Parser Parser; @@ -96,7 +99,7 @@ typedef struct Parser Parser; struct Parser { int config_file_version; - MateOutputInfo * output; + MateRROutputInfo * output; MateRRConfig * configuration; GPtrArray * outputs; GPtrArray * configurations; @@ -173,38 +176,38 @@ handle_start_element (GMarkupParseContext *context, int i; g_assert (parser->output == NULL); - parser->output = g_new0 (MateOutputInfo, 1); - parser->output->rotation = 0; + parser->output = g_object_new (MATE_TYPE_RR_OUTPUT_INFO, NULL); + parser->output->priv->rotation = 0; for (i = 0; attr_names[i] != NULL; ++i) { if (strcmp (attr_names[i], "name") == 0) { - parser->output->name = g_strdup (attr_values[i]); + parser->output->priv->name = g_strdup (attr_values[i]); break; } } - if (!parser->output->name) + if (!parser->output->priv->name) { /* This really shouldn't happen, but it's better to make * something up than to crash later. */ g_warning ("Malformed monitor configuration file"); - parser->output->name = g_strdup ("default"); + parser->output->priv->name = g_strdup ("default"); } - parser->output->connected = FALSE; - parser->output->on = FALSE; - parser->output->primary = FALSE; + parser->output->priv->connected = FALSE; + parser->output->priv->on = FALSE; + parser->output->priv->primary = FALSE; } else if (strcmp (name, "configuration") == 0) { g_assert (parser->configuration == NULL); - parser->configuration = g_new0 (MateRRConfig, 1); - parser->configuration->clone = FALSE; - parser->configuration->outputs = NULL; + parser->configuration = g_object_new (MATE_TYPE_RR_CONFIG, NULL); + parser->configuration->priv->clone = FALSE; + parser->configuration->priv->outputs = NULL; } else if (strcmp (name, "monitors") == 0) { @@ -234,8 +237,8 @@ handle_end_element (GMarkupParseContext *context, if (strcmp (name, "output") == 0) { /* If no rotation properties were set, just use MATE_RR_ROTATION_0 */ - if (parser->output->rotation == 0) - parser->output->rotation = MATE_RR_ROTATION_0; + if (parser->output->priv->rotation == 0) + parser->output->priv->rotation = MATE_RR_ROTATION_0; g_ptr_array_add (parser->outputs, parser->output); @@ -244,8 +247,8 @@ handle_end_element (GMarkupParseContext *context, else if (strcmp (name, "configuration") == 0) { g_ptr_array_add (parser->outputs, NULL); - parser->configuration->outputs = - (MateOutputInfo **)g_ptr_array_free (parser->outputs, FALSE); + parser->configuration->priv->outputs = + (MateRROutputInfo **)g_ptr_array_free (parser->outputs, FALSE); parser->outputs = g_ptr_array_new (); g_ptr_array_add (parser->configurations, parser->configuration); parser->configuration = NULL; @@ -267,96 +270,96 @@ handle_text (GMarkupParseContext *context, if (stack_is (parser, "vendor", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->connected = TRUE; + parser->output->priv->connected = TRUE; - strncpy (parser->output->vendor, text, 3); - parser->output->vendor[3] = 0; + strncpy ((gchar*) parser->output->priv->vendor, text, 3); + parser->output->priv->vendor[3] = 0; } else if (stack_is (parser, "clone", "configuration", TOPLEVEL_ELEMENT, NULL)) { if (strcmp (text, "yes") == 0) - parser->configuration->clone = TRUE; + parser->configuration->priv->clone = TRUE; } else if (stack_is (parser, "product", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->connected = TRUE; + parser->output->priv->connected = TRUE; - parser->output->product = parse_int (text); + parser->output->priv->product = parse_int (text); } else if (stack_is (parser, "serial", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->connected = TRUE; + parser->output->priv->connected = TRUE; - parser->output->serial = parse_uint (text); + parser->output->priv->serial = parse_uint (text); } else if (stack_is (parser, "width", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->on = TRUE; + parser->output->priv->on = TRUE; - parser->output->width = parse_int (text); + parser->output->priv->width = parse_int (text); } else if (stack_is (parser, "x", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->on = TRUE; + parser->output->priv->on = TRUE; - parser->output->x = parse_int (text); + parser->output->priv->x = parse_int (text); } else if (stack_is (parser, "y", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->on = TRUE; + parser->output->priv->on = TRUE; - parser->output->y = parse_int (text); + parser->output->priv->y = parse_int (text); } else if (stack_is (parser, "height", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->on = TRUE; + parser->output->priv->on = TRUE; - parser->output->height = parse_int (text); + parser->output->priv->height = parse_int (text); } else if (stack_is (parser, "rate", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { - parser->output->on = TRUE; + parser->output->priv->on = TRUE; - parser->output->rate = parse_int (text); + parser->output->priv->rate = parse_int (text); } else if (stack_is (parser, "rotation", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { if (strcmp (text, "normal") == 0) { - parser->output->rotation |= MATE_RR_ROTATION_0; + parser->output->priv->rotation |= MATE_RR_ROTATION_0; } else if (strcmp (text, "left") == 0) { - parser->output->rotation |= MATE_RR_ROTATION_90; + parser->output->priv->rotation |= MATE_RR_ROTATION_90; } else if (strcmp (text, "upside_down") == 0) { - parser->output->rotation |= MATE_RR_ROTATION_180; + parser->output->priv->rotation |= MATE_RR_ROTATION_180; } else if (strcmp (text, "right") == 0) { - parser->output->rotation |= MATE_RR_ROTATION_270; + parser->output->priv->rotation |= MATE_RR_ROTATION_270; } } else if (stack_is (parser, "reflect_x", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { if (strcmp (text, "yes") == 0) { - parser->output->rotation |= MATE_RR_REFLECT_X; + parser->output->priv->rotation |= MATE_RR_REFLECT_X; } } else if (stack_is (parser, "reflect_y", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { if (strcmp (text, "yes") == 0) { - parser->output->rotation |= MATE_RR_REFLECT_Y; + parser->output->priv->rotation |= MATE_RR_REFLECT_Y; } } else if (stack_is (parser, "primary", "output", "configuration", TOPLEVEL_ELEMENT, NULL)) { if (strcmp (text, "yes") == 0) { - parser->output->primary = TRUE; + parser->output->priv->primary = TRUE; } } else @@ -374,16 +377,16 @@ parser_free (Parser *parser) g_assert (parser != NULL); if (parser->output) - output_free (parser->output); + g_object_unref (parser->output); if (parser->configuration) - mate_rr_config_free (parser->configuration); + g_object_unref (parser->configuration); for (i = 0; i < parser->outputs->len; ++i) { - MateOutputInfo *output = parser->outputs->pdata[i]; + MateRROutputInfo *output = parser->outputs->pdata[i]; - output_free (output); + g_object_unref (output); } g_ptr_array_free (parser->outputs, TRUE); @@ -392,7 +395,7 @@ parser_free (Parser *parser) { MateRRConfig *config = parser->configurations->pdata[i]; - mate_rr_config_free (config); + g_object_unref (config); } g_ptr_array_free (parser->configurations, TRUE); @@ -443,10 +446,54 @@ out: return result; } -MateRRConfig * -mate_rr_config_new_current (MateRRScreen *screen) +static void +mate_rr_config_init (MateRRConfig *self) +{ + self->priv = G_TYPE_INSTANCE_GET_PRIVATE (self, MATE_TYPE_RR_CONFIG, MateRRConfigPrivate); + + self->priv->clone = FALSE; + self->priv->screen = NULL; + self->priv->outputs = NULL; +} + +static void +mate_rr_config_set_property (GObject *gobject, guint property_id, const GValue *value, GParamSpec *property) +{ + MateRRConfig *self = MATE_RR_CONFIG (gobject); + + switch (property_id) { + case PROP_SCREEN: + self->priv->screen = g_value_dup_object (value); + return; + default: + G_OBJECT_WARN_INVALID_PROPERTY_ID (gobject, property_id, property); + } +} + +static void +mate_rr_config_finalize (GObject *gobject) +{ + MateRRConfig *self = MATE_RR_CONFIG (gobject); + + if (self->priv->screen) + g_object_unref (self->priv->screen); + + if (self->priv->outputs) { + int i; + + for (i = 0; self->priv->outputs[i] != NULL; i++) { + MateRROutputInfo *output = self->priv->outputs[i]; + g_object_unref (output); + } + g_free (self->priv->outputs); + } + + G_OBJECT_CLASS (mate_rr_config_parent_class)->finalize (gobject); +} + +gboolean +mate_rr_config_load_current (MateRRConfig *config, GError **error) { - MateRRConfig *config; GPtrArray *a; MateRROutput **rr_outputs; int i; @@ -454,34 +501,32 @@ mate_rr_config_new_current (MateRRScreen *screen) int clone_height = -1; int last_x; - g_return_val_if_fail (screen != NULL, NULL); + g_return_val_if_fail (MATE_IS_RR_CONFIG (config), FALSE); - config = g_new0 (MateRRConfig, 1); a = g_ptr_array_new (); + rr_outputs = mate_rr_screen_list_outputs (config->priv->screen); - rr_outputs = mate_rr_screen_list_outputs (screen); - - config->clone = FALSE; + config->priv->clone = FALSE; for (i = 0; rr_outputs[i] != NULL; ++i) { MateRROutput *rr_output = rr_outputs[i]; - MateOutputInfo *output = g_new0 (MateOutputInfo, 1); + MateRROutputInfo *output = g_object_new (MATE_TYPE_RR_OUTPUT_INFO, NULL); MateRRMode *mode = NULL; const guint8 *edid_data = mate_rr_output_get_edid_data (rr_output); MateRRCrtc *crtc; - output->name = g_strdup (mate_rr_output_get_name (rr_output)); - output->connected = mate_rr_output_is_connected (rr_output); + output->priv->name = g_strdup (mate_rr_output_get_name (rr_output)); + output->priv->connected = mate_rr_output_is_connected (rr_output); - if (!output->connected) + if (!output->priv->connected) { - output->x = -1; - output->y = -1; - output->width = -1; - output->height = -1; - output->rate = -1; - output->rotation = MATE_RR_ROTATION_0; + output->priv->x = -1; + output->priv->y = -1; + output->priv->width = -1; + output->priv->height = -1; + output->priv->rate = -1; + output->priv->rotation = MATE_RR_ROTATION_0; } else { @@ -492,24 +537,24 @@ mate_rr_config_new_current (MateRRScreen *screen) if (info) { - memcpy (output->vendor, info->manufacturer_code, - sizeof (output->vendor)); + memcpy (output->priv->vendor, info->manufacturer_code, + sizeof (output->priv->vendor)); - output->product = info->product_code; - output->serial = info->serial_number; - output->aspect = info->aspect_ratio; + output->priv->product = info->product_code; + output->priv->serial = info->serial_number; + output->priv->aspect = info->aspect_ratio; } else { - strcpy (output->vendor, "???"); - output->product = 0; - output->serial = 0; + strcpy (output->priv->vendor, "???"); + output->priv->product = 0; + output->priv->serial = 0; } if (mate_rr_output_is_laptop (rr_output)) - output->display_name = g_strdup (_("Laptop")); + output->priv->display_name = g_strdup (_("Laptop")); else - output->display_name = make_display_name (info); + output->priv->display_name = make_display_name (info); g_free (info); @@ -518,28 +563,28 @@ mate_rr_config_new_current (MateRRScreen *screen) if (crtc && mode) { - output->on = TRUE; + output->priv->on = TRUE; - mate_rr_crtc_get_position (crtc, &output->x, &output->y); - output->width = mate_rr_mode_get_width (mode); - output->height = mate_rr_mode_get_height (mode); - output->rate = mate_rr_mode_get_freq (mode); - output->rotation = mate_rr_crtc_get_current_rotation (crtc); + mate_rr_crtc_get_position (crtc, &output->priv->x, &output->priv->y); + output->priv->width = mate_rr_mode_get_width (mode); + output->priv->height = mate_rr_mode_get_height (mode); + output->priv->rate = mate_rr_mode_get_freq (mode); + output->priv->rotation = mate_rr_crtc_get_current_rotation (crtc); - if (output->x == 0 && output->y == 0) { + if (output->priv->x == 0 && output->priv->y == 0) { if (clone_width == -1) { - clone_width = output->width; - clone_height = output->height; - } else if (clone_width == output->width && - clone_height == output->height) { - config->clone = TRUE; + clone_width = output->priv->width; + clone_height = output->priv->height; + } else if (clone_width == output->priv->width && + clone_height == output->priv->height) { + config->priv->clone = TRUE; } } } else { - output->on = FALSE; - config->clone = FALSE; + output->priv->on = FALSE; + config->priv->clone = FALSE; } /* Get preferred size for the monitor */ @@ -565,117 +610,152 @@ mate_rr_config_new_current (MateRRScreen *screen) if (mode) { - output->pref_width = mate_rr_mode_get_width (mode); - output->pref_height = mate_rr_mode_get_height (mode); + output->priv->pref_width = mate_rr_mode_get_width (mode); + output->priv->pref_height = mate_rr_mode_get_height (mode); } else { /* Pick some random numbers. This should basically never happen */ - output->pref_width = 1024; - output->pref_height = 768; + output->priv->pref_width = 1024; + output->priv->pref_height = 768; } } - output->primary = mate_rr_output_get_is_primary (rr_output); + output->priv->primary = mate_rr_output_get_is_primary (rr_output); g_ptr_array_add (a, output); } g_ptr_array_add (a, NULL); - config->outputs = (MateOutputInfo **)g_ptr_array_free (a, FALSE); + config->priv->outputs = (MateRROutputInfo **)g_ptr_array_free (a, FALSE); /* Walk the outputs computing the right-most edge of all * lit-up displays */ last_x = 0; - for (i = 0; config->outputs[i] != NULL; ++i) + for (i = 0; config->priv->outputs[i] != NULL; ++i) { - MateOutputInfo *output = config->outputs[i]; + MateRROutputInfo *output = config->priv->outputs[i]; - if (output->on) + if (output->priv->on) { - last_x = MAX (last_x, output->x + output->width); + last_x = MAX (last_x, output->priv->x + output->priv->width); } } /* Now position all off displays to the right of the * on displays */ - for (i = 0; config->outputs[i] != NULL; ++i) + for (i = 0; config->priv->outputs[i] != NULL; ++i) { - MateOutputInfo *output = config->outputs[i]; + MateRROutputInfo *output = config->priv->outputs[i]; - if (output->connected && !output->on) + if (output->priv->connected && !output->priv->on) { - output->x = last_x; - last_x = output->x + output->width; + output->priv->x = last_x; + last_x = output->priv->x + output->priv->width; } } g_assert (mate_rr_config_match (config, config)); - - return config; + + return TRUE; } -static void -output_free (MateOutputInfo *output) +gboolean +mate_rr_config_load_filename (MateRRConfig *result, const char *filename, GError **error) { - if (output->display_name) - g_free (output->display_name); + MateRRConfig *current; + MateRRConfig **configs; + gboolean found = FALSE; - if (output->name) - g_free (output->name); - - g_free (output); -} + g_return_val_if_fail (MATE_IS_RR_CONFIG (result), FALSE); + g_return_val_if_fail (error == NULL || *error == NULL, FALSE); -static MateOutputInfo * -output_copy (const MateOutputInfo *output) -{ - MateOutputInfo *copy = g_new0 (MateOutputInfo, 1); + if (filename == NULL) + filename = mate_rr_config_get_intended_filename (); - *copy = *output; + current = mate_rr_config_new_current (result->priv->screen, error); - copy->name = g_strdup (output->name); - copy->display_name = g_strdup (output->display_name); + configs = configurations_read_from_file (filename, error); - return copy; + if (configs) + { + int i; + + for (i = 0; configs[i] != NULL; ++i) + { + if (mate_rr_config_match (configs[i], current)) + { + int j; + GPtrArray *array; + result->priv->clone = configs[i]->priv->clone; + + array = g_ptr_array_new (); + for (j = 0; configs[i]->priv->outputs[j] != NULL; j++) { + g_ptr_array_add (array, configs[i]->priv->outputs[i]); + } + g_ptr_array_add (array, NULL); + result->priv->outputs = (MateRROutputInfo **) g_ptr_array_free (array, FALSE); + + found = TRUE; + break; + } + g_object_unref (configs[i]); + } + g_free (configs); + + if (!found) + g_set_error (error, MATE_RR_ERROR, MATE_RR_ERROR_NO_MATCHING_CONFIG, + _("none of the saved display configurations matched the active configuration")); + } + + g_object_unref (current); + return found; } static void -outputs_free (MateOutputInfo **outputs) +mate_rr_config_class_init (MateRRConfigClass *klass) { - int i; + GObjectClass *gobject_class = G_OBJECT_CLASS (klass); - g_assert (outputs != NULL); + g_type_class_add_private (klass, sizeof (MateRROutputInfoPrivate)); - for (i = 0; outputs[i] != NULL; ++i) - output_free (outputs[i]); + gobject_class->set_property = mate_rr_config_set_property; + gobject_class->finalize = mate_rr_config_finalize; - g_free (outputs); + g_object_class_install_property (gobject_class, PROP_SCREEN, + g_param_spec_object ("screen", "Screen", "The MateRRScreen this config applies to", MATE_TYPE_RR_SCREEN, + G_PARAM_WRITABLE | G_PARAM_CONSTRUCT_ONLY | G_PARAM_STATIC_NICK | G_PARAM_STATIC_BLURB)); } -void -mate_rr_config_free (MateRRConfig *config) +MateRRConfig * +mate_rr_config_new_current (MateRRScreen *screen, GError **error) { - g_return_if_fail (config != NULL); - outputs_free (config->outputs); - - g_free (config); + MateRRConfig *self = g_object_new (MATE_TYPE_RR_CONFIG, "screen", screen, NULL); + + if (mate_rr_config_load_current (self, error)) + return self; + else + { + g_object_unref (self); + return NULL; + } } -static void -configurations_free (MateRRConfig **configurations) +MateRRConfig * +mate_rr_config_new_stored (MateRRScreen *screen, GError **error) { - int i; - - g_assert (configurations != NULL); + MateRRConfig *self = g_object_new (MATE_TYPE_RR_CONFIG, "screen", screen, NULL); - for (i = 0; configurations[i] != NULL; ++i) - mate_rr_config_free (configurations[i]); - - g_free (configurations); + if (mate_rr_config_load_filename (self, NULL, error)) + return self; + else + { + g_object_unref (self); + return NULL; + } } static gboolean @@ -720,75 +800,75 @@ out: } static gboolean -output_match (MateOutputInfo *output1, MateOutputInfo *output2) +output_match (MateRROutputInfo *output1, MateRROutputInfo *output2) { - g_assert (output1 != NULL); - g_assert (output2 != NULL); + g_assert (MATE_IS_RR_OUTPUT_INFO (output1)); + g_assert (MATE_IS_RR_OUTPUT_INFO (output2)); - if (strcmp (output1->name, output2->name) != 0) + if (strcmp (output1->priv->name, output2->priv->name) != 0) return FALSE; - if (strcmp (output1->vendor, output2->vendor) != 0) + if (strcmp (output1->priv->vendor, output2->priv->vendor) != 0) return FALSE; - if (output1->product != output2->product) + if (output1->priv->product != output2->priv->product) return FALSE; - if (output1->serial != output2->serial) + if (output1->priv->serial != output2->priv->serial) return FALSE; - if (output1->connected != output2->connected) + if (output1->priv->connected != output2->priv->connected) return FALSE; return TRUE; } static gboolean -output_equal (MateOutputInfo *output1, MateOutputInfo *output2) +output_equal (MateRROutputInfo *output1, MateRROutputInfo *output2) { - g_assert (output1 != NULL); - g_assert (output2 != NULL); + g_assert (MATE_IS_RR_OUTPUT_INFO (output1)); + g_assert (MATE_IS_RR_OUTPUT_INFO (output2)); if (!output_match (output1, output2)) return FALSE; - if (output1->on != output2->on) + if (output1->priv->on != output2->priv->on) return FALSE; - if (output1->on) + if (output1->priv->on) { - if (output1->width != output2->width) + if (output1->priv->width != output2->priv->width) return FALSE; - if (output1->height != output2->height) + if (output1->priv->height != output2->priv->height) return FALSE; - if (output1->rate != output2->rate) + if (output1->priv->rate != output2->priv->rate) return FALSE; - if (output1->x != output2->x) + if (output1->priv->x != output2->priv->x) return FALSE; - if (output1->y != output2->y) + if (output1->priv->y != output2->priv->y) return FALSE; - if (output1->rotation != output2->rotation) + if (output1->priv->rotation != output2->priv->rotation) return FALSE; } return TRUE; } -static MateOutputInfo * +static MateRROutputInfo * find_output (MateRRConfig *config, const char *name) { int i; - for (i = 0; config->outputs[i] != NULL; ++i) + for (i = 0; config->priv->outputs[i] != NULL; ++i) { - MateOutputInfo *output = config->outputs[i]; + MateRROutputInfo *output = config->priv->outputs[i]; - if (strcmp (name, output->name) == 0) + if (strcmp (name, output->priv->name) == 0) return output; } @@ -802,13 +882,15 @@ gboolean mate_rr_config_match (MateRRConfig *c1, MateRRConfig *c2) { int i; + g_return_val_if_fail (MATE_IS_RR_CONFIG (c1), FALSE); + g_return_val_if_fail (MATE_IS_RR_CONFIG (c2), FALSE); - for (i = 0; c1->outputs[i] != NULL; ++i) + for (i = 0; c1->priv->outputs[i] != NULL; ++i) { - MateOutputInfo *output1 = c1->outputs[i]; - MateOutputInfo *output2; + MateRROutputInfo *output1 = c1->priv->outputs[i]; + MateRROutputInfo *output2; - output2 = find_output (c2, output1->name); + output2 = find_output (c2, output1->priv->name); if (!output2 || !output_match (output1, output2)) return FALSE; } @@ -824,13 +906,15 @@ mate_rr_config_equal (MateRRConfig *c1, MateRRConfig *c2) { int i; + g_return_val_if_fail (MATE_IS_RR_CONFIG (c1), FALSE); + g_return_val_if_fail (MATE_IS_RR_CONFIG (c2), FALSE); - for (i = 0; c1->outputs[i] != NULL; ++i) + for (i = 0; c1->priv->outputs[i] != NULL; ++i) { - MateOutputInfo *output1 = c1->outputs[i]; - MateOutputInfo *output2; + MateRROutputInfo *output1 = c1->priv->outputs[i]; + MateRROutputInfo *output2; - output2 = find_output (c2, output1->name); + output2 = find_output (c2, output1->priv->name); if (!output2 || !output_equal (output1, output2)) return FALSE; } @@ -838,34 +922,39 @@ mate_rr_config_equal (MateRRConfig *c1, return TRUE; } -static MateOutputInfo ** +static MateRROutputInfo ** make_outputs (MateRRConfig *config) { GPtrArray *outputs; - MateOutputInfo *first_on; + MateRROutputInfo *first_on; int i; outputs = g_ptr_array_new (); first_on = NULL; - for (i = 0; config->outputs[i] != NULL; ++i) + for (i = 0; config->priv->outputs[i] != NULL; ++i) { - MateOutputInfo *old = config->outputs[i]; - MateOutputInfo *new = output_copy (old); - - if (old->on && !first_on) + MateRROutputInfo *old = config->priv->outputs[i]; + MateRROutputInfo *new = g_object_new (MATE_TYPE_RR_OUTPUT_INFO, NULL); + *(new->priv) = *(old->priv); + if (old->priv->name) + new->priv->name = g_strdup (old->priv->name); + if (old->priv->display_name) + new->priv->display_name = g_strdup (old->priv->display_name); + + if (old->priv->on && !first_on) first_on = old; - if (config->clone && new->on) + if (config->priv->clone && new->priv->on) { g_assert (first_on); - new->width = first_on->width; - new->height = first_on->height; - new->rotation = first_on->rotation; - new->x = 0; - new->y = 0; + new->priv->width = first_on->priv->width; + new->priv->height = first_on->priv->height; + new->priv->rotation = first_on->priv->rotation; + new->priv->x = 0; + new->priv->y = 0; } g_ptr_array_add (outputs, new); @@ -873,7 +962,7 @@ make_outputs (MateRRConfig *config) g_ptr_array_add (outputs, NULL); - return (MateOutputInfo **)g_ptr_array_free (outputs, FALSE); + return (MateRROutputInfo **)g_ptr_array_free (outputs, FALSE); } gboolean @@ -881,11 +970,13 @@ mate_rr_config_applicable (MateRRConfig *configuration, MateRRScreen *screen, GError **error) { - MateOutputInfo **outputs; + MateRROutputInfo **outputs; CrtcAssignment *assign; gboolean result; + int i; - g_return_val_if_fail (configuration != NULL, FALSE); + g_return_val_if_fail (MATE_IS_RR_CONFIG (configuration), FALSE); + g_return_val_if_fail (MATE_IS_RR_SCREEN (screen), FALSE); g_return_val_if_fail (error == NULL || *error == NULL, FALSE); outputs = make_outputs (configuration); @@ -901,7 +992,9 @@ mate_rr_config_applicable (MateRRConfig *configuration, result = FALSE; } - outputs_free (outputs); + for (i = 0; outputs[i] != NULL; i++) { + g_object_unref (outputs[i]); + } return result; } @@ -969,46 +1062,46 @@ emit_configuration (MateRRConfig *config, g_string_append_printf (string, " \n"); - g_string_append_printf (string, " %s\n", yes_no (config->clone)); + g_string_append_printf (string, " %s\n", yes_no (config->priv->clone)); - for (j = 0; config->outputs[j] != NULL; ++j) + for (j = 0; config->priv->outputs[j] != NULL; ++j) { - MateOutputInfo *output = config->outputs[j]; + MateRROutputInfo *output = config->priv->outputs[j]; g_string_append_printf ( - string, " \n", output->name); + string, " \n", output->priv->name); - if (output->connected && *output->vendor != '\0') + if (output->priv->connected && *output->priv->vendor != '\0') { g_string_append_printf ( - string, " %s\n", output->vendor); + string, " %s\n", output->priv->vendor); g_string_append_printf ( - string, " 0x%04x\n", output->product); + string, " 0x%04x\n", output->priv->product); g_string_append_printf ( - string, " 0x%08x\n", output->serial); + string, " 0x%08x\n", output->priv->serial); } /* An unconnected output which is on does not make sense */ - if (output->connected && output->on) + if (output->priv->connected && output->priv->on) { g_string_append_printf ( - string, " %d\n", output->width); + string, " %d\n", output->priv->width); g_string_append_printf ( - string, " %d\n", output->height); + string, " %d\n", output->priv->height); g_string_append_printf ( - string, " %d\n", output->rate); + string, " %d\n", output->priv->rate); g_string_append_printf ( - string, " %d\n", output->x); + string, " %d\n", output->priv->x); g_string_append_printf ( - string, " %d\n", output->y); + string, " %d\n", output->priv->y); g_string_append_printf ( - string, " %s\n", get_rotation_name (output->rotation)); + string, " %s\n", get_rotation_name (output->priv->rotation)); g_string_append_printf ( - string, " %s\n", get_reflect_x (output->rotation)); + string, " %s\n", get_reflect_x (output->priv->rotation)); g_string_append_printf ( - string, " %s\n", get_reflect_y (output->rotation)); + string, " %s\n", get_reflect_y (output->priv->rotation)); g_string_append_printf ( - string, " %s\n", yes_no (output->primary)); + string, " %s\n", yes_no (output->priv->primary)); } g_string_append_printf (string, " \n"); @@ -1028,37 +1121,37 @@ mate_rr_config_sanitize (MateRRConfig *config) * make sure the configuration starts at (0, 0) */ x_offset = y_offset = G_MAXINT; - for (i = 0; config->outputs[i]; ++i) + for (i = 0; config->priv->outputs[i]; ++i) { - MateOutputInfo *output = config->outputs[i]; + MateRROutputInfo *output = config->priv->outputs[i]; - if (output->on) + if (output->priv->on) { - x_offset = MIN (x_offset, output->x); - y_offset = MIN (y_offset, output->y); + x_offset = MIN (x_offset, output->priv->x); + y_offset = MIN (y_offset, output->priv->y); } } - for (i = 0; config->outputs[i]; ++i) + for (i = 0; config->priv->outputs[i]; ++i) { - MateOutputInfo *output = config->outputs[i]; + MateRROutputInfo *output = config->priv->outputs[i]; - if (output->on) + if (output->priv->on) { - output->x -= x_offset; - output->y -= y_offset; + output->priv->x -= x_offset; + output->priv->y -= y_offset; } } /* Only one primary, please */ found = FALSE; - for (i = 0; config->outputs[i]; ++i) + for (i = 0; config->priv->outputs[i]; ++i) { - if (config->outputs[i]->primary) + if (config->priv->outputs[i]->priv->primary) { if (found) { - config->outputs[i]->primary = FALSE; + config->priv->outputs[i]->priv->primary = FALSE; } else { @@ -1069,13 +1162,13 @@ mate_rr_config_sanitize (MateRRConfig *config) } static gboolean -output_info_is_laptop (MateOutputInfo *info) +output_info_is_laptop (MateRROutputInfo *info) { - if (info->name - && (strstr (info->name, "lvds") || /* Most drivers use an "LVDS" prefix... */ - strstr (info->name, "LVDS") || - strstr (info->name, "Lvds") || - strstr (info->name, "LCD"))) /* ... but fglrx uses "LCD" in some versions. Shoot me now, kthxbye. */ + if (info->priv->name + && (strstr (info->priv->name, "lvds") || /* Most drivers use an "LVDS" prefix... */ + strstr (info->priv->name, "LVDS") || + strstr (info->priv->name, "Lvds") || + strstr (info->priv->name, "LCD"))) /* ... but fglrx uses "LCD" in some versions. Shoot me now, kthxbye. */ return TRUE; return FALSE; @@ -1084,35 +1177,39 @@ output_info_is_laptop (MateOutputInfo *info) gboolean mate_rr_config_ensure_primary (MateRRConfig *configuration) { - int i; - MateOutputInfo *laptop; - MateOutputInfo *top_left; - gboolean found; + int i; + MateRROutputInfo *laptop; + MateRROutputInfo *top_left; + gboolean found; + MateRRConfigPrivate *priv; + + g_return_val_if_fail (MATE_IS_RR_CONFIG (configuration), FALSE); laptop = NULL; top_left = NULL; found = FALSE; + priv = configuration->priv; - for (i = 0; configuration->outputs[i] != NULL; ++i) { - MateOutputInfo *info = configuration->outputs[i]; + for (i = 0; priv->outputs[i] != NULL; ++i) { + MateRROutputInfo *info = priv->outputs[i]; - if (! info->on) { - info->primary = FALSE; + if (! info->priv->on) { + info->priv->primary = FALSE; continue; } /* ensure only one */ - if (info->primary) { + if (info->priv->primary) { if (found) { - info->primary = FALSE; + info->priv->primary = FALSE; } else { found = TRUE; } } if (top_left == NULL - || (info->x < top_left->x - && info->y < top_left->y)) { + || (info->priv->x < top_left->priv->x + && info->priv->y < top_left->priv->y)) { top_left = info; } if (laptop == NULL @@ -1123,11 +1220,11 @@ mate_rr_config_ensure_primary (MateRRConfig *configuration) } } - if (! found) { + if (!found) { if (laptop != NULL) { - laptop->primary = TRUE; + laptop->priv->primary = TRUE; } else { - top_left->primary = TRUE; + top_left->priv->primary = TRUE; } } @@ -1144,7 +1241,7 @@ mate_rr_config_save (MateRRConfig *configuration, GError **error) gchar *backup_filename; gboolean result; - g_return_val_if_fail (configuration != NULL, FALSE); + g_return_val_if_fail (MATE_IS_RR_CONFIG (configuration), FALSE); g_return_val_if_fail (error == NULL || *error == NULL, FALSE); output = g_string_new (""); @@ -1162,9 +1259,10 @@ mate_rr_config_save (MateRRConfig *configuration, GError **error) { if (!mate_rr_config_match (configurations[i], configuration)) emit_configuration (configurations[i], output); + g_object_unref (configurations[i]); } - configurations_free (configurations); + g_free (configurations); } emit_configuration (configuration, output); @@ -1186,89 +1284,6 @@ mate_rr_config_save (MateRRConfig *configuration, GError **error) return result; } -static MateRRConfig * -mate_rr_config_copy (const MateRRConfig *config) -{ - MateRRConfig *copy = g_new0 (MateRRConfig, 1); - int i; - GPtrArray *array = g_ptr_array_new (); - - copy->clone = config->clone; - - for (i = 0; config->outputs[i] != NULL; ++i) - g_ptr_array_add (array, output_copy (config->outputs[i])); - - g_ptr_array_add (array, NULL); - copy->outputs = (MateOutputInfo **)g_ptr_array_free (array, FALSE); - - return copy; -} - -static MateRRConfig * -config_new_stored (MateRRScreen *screen, const char *filename, GError **error) -{ - MateRRConfig *current; - MateRRConfig **configs; - MateRRConfig *result; - - g_return_val_if_fail (screen != NULL, NULL); - g_return_val_if_fail (error == NULL || *error == NULL, NULL); - - current = mate_rr_config_new_current (screen); - - configs = configurations_read_from_file (filename, error); - - result = NULL; - if (configs) - { - int i; - - for (i = 0; configs[i] != NULL; ++i) - { - if (mate_rr_config_match (configs[i], current)) - { - result = mate_rr_config_copy (configs[i]); - break; - } - } - - if (result == NULL) - g_set_error (error, MATE_RR_ERROR, MATE_RR_ERROR_NO_MATCHING_CONFIG, - _("none of the saved display configurations matched the active configuration")); - - configurations_free (configs); - } - - mate_rr_config_free (current); - - return result; -} - -MateRRConfig * -mate_rr_config_new_stored (MateRRScreen *screen, GError **error) -{ - char *intended_filename; - MateRRConfig *config; - - intended_filename = mate_rr_config_get_intended_filename (); - - config = config_new_stored (screen, intended_filename, error); - - g_free (intended_filename); - - return config; -} - -#ifndef MATE_DISABLE_DEPRECATED_SOURCE -gboolean -mate_rr_config_apply (MateRRConfig *config, - MateRRScreen *screen, - GError **error) -{ - return mate_rr_config_apply_with_time (config, screen, GDK_CURRENT_TIME, error); -} -#endif - gboolean mate_rr_config_apply_with_time (MateRRConfig *config, MateRRScreen *screen, @@ -1276,14 +1291,20 @@ mate_rr_config_apply_with_time (MateRRConfig *config, GError **error) { CrtcAssignment *assignment; - MateOutputInfo **outputs; + MateRROutputInfo **outputs; gboolean result = FALSE; + int i; + + g_return_val_if_fail (MATE_IS_RR_CONFIG (config), FALSE); + g_return_val_if_fail (MATE_IS_RR_SCREEN (screen), FALSE); outputs = make_outputs (config); assignment = crtc_assignment_new (screen, outputs, error); - outputs_free (outputs); + for (i = 0; outputs[i] != NULL; i++) + g_object_unref (outputs[i]); + g_free (outputs); if (assignment) { @@ -1298,74 +1319,6 @@ mate_rr_config_apply_with_time (MateRRConfig *config, return result; } -#ifndef MATE_DISABLE_DEPRECATED_SOURCE -/** - * mate_rr_config_apply_stored: - * @screen: A #MateRRScreen - * @error: Location to store error, or %NULL - * - * See the documentation for mate_rr_config_apply_from_filename(). This - * function simply calls that other function with a filename of - * mate_rr_config_get_intended_filename(). - * - * Deprecated: 2.26: Use mate_rr_config_apply_from_filename() instead and pass it - * the filename from mate_rr_config_get_intended_filename(). - */ -gboolean -mate_rr_config_apply_stored (MateRRScreen *screen, GError **error) -{ - char *filename; - gboolean result; - - filename = mate_rr_config_get_intended_filename (); - result = mate_rr_config_apply_from_filename_with_time (screen, filename, GDK_CURRENT_TIME, error); - g_free (filename); - - return result; -} -#endif - -#ifndef MATE_DISABLE_DEPRECATED_SOURCE -/* mate_rr_config_apply_from_filename: - * @screen: A #MateRRScreen - * @filename: Path of the file to look in for stored RANDR configurations. - * @error: Location to store error, or %NULL - * - * First, this function refreshes the @screen to match the current RANDR - * configuration from the X server. Then, it tries to load the file in - * @filename and looks for suitable matching RANDR configurations in the file; - * if one is found, that configuration will be applied to the current set of - * RANDR outputs. - * - * Typically, @filename is the result of mate_rr_config_get_intended_filename() or - * mate_rr_config_get_backup_filename(). - * - * Returns: TRUE if the RANDR configuration was loaded and applied from - * $(XDG_CONFIG_HOME)/monitors.xml, or FALSE otherwise: - * - * If the current RANDR configuration could not be refreshed, the @error will - * have a domain of #MATE_RR_ERROR and a corresponding error code. - * - * If the file in question is loaded successfully but the configuration cannot - * be applied, the @error will have a domain of #MATE_RR_ERROR. Note that an - * error code of #MATE_RR_ERROR_NO_MATCHING_CONFIG is not a real error; it - * simply means that there were no stored configurations that match the current - * set of RANDR outputs. - * - * If the file in question cannot be loaded, the @error will have a domain of - * #G_FILE_ERROR. Note that an error code of G_FILE_ERROR_NOENT is not really - * an error, either; it means that there was no stored configuration file and so - * nothing is changed. - * - * @Deprecated: 2.28: use mate_rr_config_apply_from_filename_with_time() instead. - */ -gboolean -mate_rr_config_apply_from_filename (MateRRScreen *screen, const char *filename, GError **error) -{ - return mate_rr_config_apply_from_filename_with_time (screen, filename, GDK_CURRENT_TIME, error); -} -#endif - /* mate_rr_config_apply_from_filename_with_time: * @screen: A #MateRRScreen * @filename: Path of the file to look in for stored RANDR configurations. @@ -1404,7 +1357,7 @@ mate_rr_config_apply_from_filename_with_time (MateRRScreen *screen, const char * MateRRConfig *stored; GError *my_error; - g_return_val_if_fail (screen != NULL, FALSE); + g_return_val_if_fail (MATE_IS_RR_SCREEN (screen), FALSE); g_return_val_if_fail (filename != NULL, FALSE); g_return_val_if_fail (error == NULL || *error == NULL, FALSE); @@ -1418,25 +1371,63 @@ mate_rr_config_apply_from_filename_with_time (MateRRScreen *screen, const char * /* This means the screen didn't change, so just proceed */ } - stored = config_new_stored (screen, filename, error); + stored = g_object_new (MATE_TYPE_RR_CONFIG, "screen", screen, NULL); - if (stored) + if (mate_rr_config_load_filename (stored, filename, error)) { gboolean result; mate_rr_config_ensure_primary (stored); result = mate_rr_config_apply_with_time (stored, screen, timestamp, error); - mate_rr_config_free (stored); + g_object_unref (stored); return result; } else { + g_object_unref (stored); return FALSE; } } +/** + * mate_rr_config_get_outputs: + * + * Returns: (array zero-terminated=1) (element-type MateDesktop.RROutputInfo) (transfer none): the output configuration for this #MateRRConfig + */ +MateRROutputInfo ** +mate_rr_config_get_outputs (MateRRConfig *self) +{ + g_return_val_if_fail (MATE_IS_RR_CONFIG (self), NULL); + + return self->priv->outputs; +} + +/** + * mate_rr_config_get_clone: + * + * Returns: whether at least two outputs are at (0, 0) offset and they + * have the same width/height. Those outputs are of course connected and on + * (i.e. they have a CRTC assigned). + */ +gboolean +mate_rr_config_get_clone (MateRRConfig *self) +{ + g_return_val_if_fail (MATE_IS_RR_CONFIG (self), FALSE); + + return self->priv->clone; +} + +void +mate_rr_config_set_clone (MateRRConfig *self, gboolean clone) +{ + g_return_if_fail (MATE_IS_RR_CONFIG (self)); + + self->priv->clone = clone; +} + + /* * CRTC assignment */ @@ -1679,12 +1670,12 @@ accumulate_error (GString *accumulated_error, GError *error) */ static gboolean real_assign_crtcs (MateRRScreen *screen, - MateOutputInfo **outputs, + MateRROutputInfo **outputs, CrtcAssignment *assignment, GError **error) { MateRRCrtc **crtcs = mate_rr_screen_list_crtcs (screen); - MateOutputInfo *output; + MateRROutputInfo *output; int i; gboolean tried_mode; GError *my_error; @@ -1696,7 +1687,7 @@ real_assign_crtcs (MateRRScreen *screen, return TRUE; /* It is always allowed for an output to be turned off */ - if (!output->on) + if (!output->priv->on) { return real_assign_crtcs (screen, outputs + 1, assignment, error); } @@ -1720,7 +1711,7 @@ real_assign_crtcs (MateRRScreen *screen, */ for (pass = 0; pass < 2; ++pass) { - MateRROutput *mate_rr_output = mate_rr_screen_get_output_by_name (screen, output->name); + MateRROutput *mate_rr_output = mate_rr_screen_get_output_by_name (screen, output->priv->name); MateRRMode **modes = mate_rr_output_list_modes (mate_rr_output); int j; @@ -1739,21 +1730,21 @@ real_assign_crtcs (MateRRScreen *screen, _("CRTC %d: trying mode %dx%d@%dHz with output at %dx%d@%dHz (pass %d)\n"), crtc_id, mode_width, mode_height, mode_freq, - output->width, output->height, output->rate, + output->priv->width, output->priv->height, output->priv->rate, pass); - if (mode_width == output->width && - mode_height == output->height && - (pass == 1 || mode_freq == output->rate)) + if (mode_width == output->priv->width && + mode_height == output->priv->height && + (pass == 1 || mode_freq == output->priv->rate)) { tried_mode = TRUE; my_error = NULL; if (crtc_assignment_assign ( assignment, crtc, modes[j], - output->x, output->y, - output->rotation, - output->primary, + output->priv->x, output->priv->y, + output->priv->rotation, + output->priv->primary, mate_rr_output, &my_error)) { @@ -1841,7 +1832,7 @@ get_required_virtual_size (CrtcAssignment *assign, int *width, int *height) } static CrtcAssignment * -crtc_assignment_new (MateRRScreen *screen, MateOutputInfo **outputs, GError **error) +crtc_assignment_new (MateRRScreen *screen, MateRROutputInfo **outputs, GError **error) { CrtcAssignment *assignment = g_new0 (CrtcAssignment, 1); -- cgit v1.2.1