display: Simplify add_key() calls and function
This commit is contained in:
parent
dd7c5fe67a
commit
6b6e17788e
1 changed files with 7 additions and 10 deletions
|
@ -330,16 +330,13 @@ foreach (GtkTreeModel *model,
|
||||||
}
|
}
|
||||||
|
|
||||||
static void
|
static void
|
||||||
add_key (GtkWidget *widget,
|
add_key (GtkTreeModel *model,
|
||||||
const char *text,
|
const char *text,
|
||||||
gboolean preferred,
|
gboolean preferred,
|
||||||
int width, int height, int rate,
|
int width, int height, int rate,
|
||||||
GnomeRRRotation rotation)
|
GnomeRRRotation rotation)
|
||||||
{
|
{
|
||||||
ForeachInfo info;
|
ForeachInfo info;
|
||||||
GtkComboBox *box = GTK_COMBO_BOX (widget);
|
|
||||||
GtkTreeModel *model = gtk_combo_box_get_model (box);
|
|
||||||
GtkListStore *store = GTK_LIST_STORE (model);
|
|
||||||
|
|
||||||
info.text = text;
|
info.text = text;
|
||||||
info.found = FALSE;
|
info.found = FALSE;
|
||||||
|
@ -350,7 +347,7 @@ add_key (GtkWidget *widget,
|
||||||
{
|
{
|
||||||
GtkTreeIter iter;
|
GtkTreeIter iter;
|
||||||
g_debug ("adding %s with rate %d Hz", text, rate);
|
g_debug ("adding %s with rate %d Hz", text, rate);
|
||||||
gtk_list_store_insert_with_values (store, &iter, -1,
|
gtk_list_store_insert_with_values (GTK_LIST_STORE (model), &iter, -1,
|
||||||
TEXT_COL, text,
|
TEXT_COL, text,
|
||||||
WIDTH_COL, width,
|
WIDTH_COL, width,
|
||||||
HEIGHT_COL, height,
|
HEIGHT_COL, height,
|
||||||
|
@ -365,7 +362,7 @@ add_key (GtkWidget *widget,
|
||||||
if (preferred)
|
if (preferred)
|
||||||
{
|
{
|
||||||
g_debug ("replacing %s with rate %d Hz (preferred mode)", text, rate);
|
g_debug ("replacing %s with rate %d Hz (preferred mode)", text, rate);
|
||||||
gtk_list_store_set (store, &info.iter,
|
gtk_list_store_set (GTK_LIST_STORE (model), &info.iter,
|
||||||
RATE_COL, rate,
|
RATE_COL, rate,
|
||||||
-1);
|
-1);
|
||||||
return;
|
return;
|
||||||
|
@ -374,7 +371,7 @@ add_key (GtkWidget *widget,
|
||||||
{
|
{
|
||||||
int old_rate;
|
int old_rate;
|
||||||
|
|
||||||
gtk_tree_model_get (GTK_TREE_MODEL (store), &info.iter,
|
gtk_tree_model_get (model, &info.iter,
|
||||||
RATE_COL, &old_rate,
|
RATE_COL, &old_rate,
|
||||||
-1);
|
-1);
|
||||||
|
|
||||||
|
@ -382,7 +379,7 @@ add_key (GtkWidget *widget,
|
||||||
if (rate > old_rate)
|
if (rate > old_rate)
|
||||||
{
|
{
|
||||||
g_debug ("replacing %s with rate %d Hz (old rate: %d)", text, rate, old_rate);
|
g_debug ("replacing %s with rate %d Hz (old rate: %d)", text, rate, old_rate);
|
||||||
gtk_list_store_set (store, &info.iter,
|
gtk_list_store_set (GTK_LIST_STORE (model), &info.iter,
|
||||||
RATE_COL, rate,
|
RATE_COL, rate,
|
||||||
-1);
|
-1);
|
||||||
return;
|
return;
|
||||||
|
@ -412,7 +409,7 @@ add_mode (CcDisplayPanel *self,
|
||||||
|
|
||||||
preferred = (gnome_rr_mode_get_id (mode) == preferred_id);
|
preferred = (gnome_rr_mode_get_id (mode) == preferred_id);
|
||||||
text = make_resolution_string (width, height);
|
text = make_resolution_string (width, height);
|
||||||
add_key (self->priv->resolution_combo,
|
add_key (gtk_combo_box_get_model (GTK_COMBO_BOX (self->priv->resolution_combo)),
|
||||||
text, preferred, width, height, rate, -1);
|
text, preferred, width, height, rate, -1);
|
||||||
g_free (text);
|
g_free (text);
|
||||||
}
|
}
|
||||||
|
@ -501,7 +498,7 @@ rebuild_rotation_combo (CcDisplayPanel *self)
|
||||||
/* NULL-GError --- FIXME: we should say why this rotation is not available! */
|
/* NULL-GError --- FIXME: we should say why this rotation is not available! */
|
||||||
if (gnome_rr_config_applicable (self->priv->current_configuration, self->priv->screen, NULL))
|
if (gnome_rr_config_applicable (self->priv->current_configuration, self->priv->screen, NULL))
|
||||||
{
|
{
|
||||||
add_key (self->priv->rotation_combo, _(info->name), FALSE, 0, 0, 0, info->rotation);
|
add_key (gtk_combo_box_get_model (GTK_COMBO_BOX (self->priv->rotation_combo)), _(info->name), FALSE, 0, 0, 0, info->rotation);
|
||||||
|
|
||||||
if (info->rotation == current)
|
if (info->rotation == current)
|
||||||
selection = _(info->name);
|
selection = _(info->name);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue