[evolution-activesync] Fix some warnings
- From: dwmw2 <dwmw2 src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution-activesync] Fix some warnings
- Date: Thu, 11 Jun 2015 16:01:29 +0000 (UTC)
commit 32e12834e7b61e8503083800c849846f2fad4bb3
Author: Oliver Luo <lyc pku eecs gmail com>
Date: Sat Jun 6 23:50:28 2015 +0800
Fix some warnings
Fix some warnings. Most are deprecated usage of glib and gtk.
collection/e-eas-backend.c | 6 +-
configuration/e-mail-config-eas-backend.c | 115 +++++++++++++++--------------
2 files changed, 63 insertions(+), 58 deletions(-)
---
diff --git a/collection/e-eas-backend.c b/collection/e-eas-backend.c
index 563e3b1..5486981 100644
--- a/collection/e-eas-backend.c
+++ b/collection/e-eas-backend.c
@@ -36,7 +36,7 @@ struct _EEasBackendPrivate {
GHashTable *folders;
gchar *sync_state;
- GMutex *sync_state_lock;
+ GMutex sync_state_lock;
};
struct _SyncFoldersClosure {
@@ -74,7 +74,7 @@ eas_backend_finalize (GObject *object)
g_hash_table_destroy (priv->folders);
g_free (priv->sync_state);
- g_mutex_free (priv->sync_state_lock);
+ g_mutex_clear (&priv->sync_state_lock);
/* Chain up to parent's finalize() method */
G_OBJECT_CLASS (e_eas_backend_parent_class)->finalize (object);
@@ -110,7 +110,7 @@ e_eas_backend_init (EEasBackend *backend)
(GDestroyNotify) g_free,
(GDestroyNotify) g_object_unref);
- backend->priv->sync_state_lock = g_mutex_new ();
+ g_mutex_init (&backend->priv->sync_state_lock);
}
void
diff --git a/configuration/e-mail-config-eas-backend.c b/configuration/e-mail-config-eas-backend.c
index 3608e48..5602e85 100644
--- a/configuration/e-mail-config-eas-backend.c
+++ b/configuration/e-mail-config-eas-backend.c
@@ -94,33 +94,35 @@ discover_server_url (GtkWidget *button, EMailConfigServiceBackend *backend)
page = e_mail_config_service_backend_get_page (backend);
email_address = e_mail_config_service_page_get_email_address (page);
- handler = eas_mail_handler_new(email_address, &error);
- if (error) {
- g_warning ("Unable to create mailHandler. We don't suppport auto-discover: %s\n",
error->message);
- g_error_free (error);
- gtk_widget_set_sensitive (button, FALSE);
- return;
- }
+ if (email_address != NULL) {
+ handler = eas_mail_handler_new(email_address, &error);
+ if (error) {
+ g_warning ("Unable to create mailHandler. We don't suppport auto-discover: %s\n",
error->message);
+ g_error_free (error);
+ gtk_widget_set_sensitive (button, FALSE);
+ return;
+ }
- username = g_strdup (gtk_entry_get_text(username_entry));
- if (username == NULL || *username == '\0' || strcmp (username, email_address) == 0) {
- g_free (username);
- username = NULL;
- }
+ username = g_strdup (gtk_entry_get_text((GtkEntry *)username_entry));
+ if (username == NULL || *username == '\0' || strcmp (username, email_address) == 0) {
+ g_free (username);
+ username = NULL;
+ }
- eas_mail_handler_autodiscover(
- handler,
- email_address,
- username,
- &uri,
- NULL,
- &error);
+ eas_mail_handler_autodiscover(
+ handler,
+ email_address,
+ username,
+ &uri,
+ NULL,
+ &error);
- if (!error && uri && uri[0])
- gtk_entry_set_text ((GtkEntry *)host_entry, uri);
+ if (!error && uri && uri[0])
+ gtk_entry_set_text ((GtkEntry *)host_entry, uri);
- g_free (username);
- g_object_unref (handler);
+ g_free (username);
+ g_object_unref (handler);
+ }
}
static void
@@ -158,13 +160,14 @@ mail_config_eas_backend_insert_widgets (EMailConfigServiceBackend *backend,
markup = g_markup_printf_escaped ("<b>%s</b>", text);
widget = gtk_label_new (markup);
gtk_label_set_use_markup (GTK_LABEL (widget), TRUE);
- gtk_misc_set_alignment (GTK_MISC (widget), 0.0, 0.5);
+ gtk_widget_set_halign (widget, GTK_ALIGN_START);
+ gtk_widget_set_valign (widget, GTK_ALIGN_CENTER);
gtk_box_pack_start (GTK_BOX (parent), widget, FALSE, FALSE, 0);
gtk_widget_show (widget);
g_free (markup);
widget = gtk_grid_new ();
- gtk_widget_set_margin_left (widget, 12);
+ gtk_widget_set_margin_start (widget, 12);
gtk_grid_set_row_spacing (GTK_GRID (widget), 6);
gtk_grid_set_column_spacing (GTK_GRID (widget), 6);
gtk_box_pack_start (GTK_BOX (parent), widget, FALSE, FALSE, 0);
@@ -173,7 +176,8 @@ mail_config_eas_backend_insert_widgets (EMailConfigServiceBackend *backend,
container = widget;
widget = gtk_label_new_with_mnemonic (_("User_name:"));
- gtk_misc_set_alignment (GTK_MISC (widget), 1.0, 0.5);
+ gtk_widget_set_halign (widget, GTK_ALIGN_END);
+ gtk_widget_set_valign (widget, GTK_ALIGN_CENTER);
gtk_grid_attach (GTK_GRID (container), widget, 0, 0, 1, 1);
gtk_widget_show (widget);
@@ -187,7 +191,8 @@ mail_config_eas_backend_insert_widgets (EMailConfigServiceBackend *backend,
gtk_widget_show (widget);
widget = gtk_label_new_with_mnemonic (_("_Server URL:"));
- gtk_misc_set_alignment (GTK_MISC (widget), 1.0, 0.5);
+ gtk_widget_set_halign (widget, GTK_ALIGN_END);
+ gtk_widget_set_valign (widget, GTK_ALIGN_CENTER);
gtk_grid_attach (GTK_GRID (container), widget, 0, 1, 1, 1);
gtk_widget_show (widget);
@@ -205,13 +210,14 @@ mail_config_eas_backend_insert_widgets (EMailConfigServiceBackend *backend,
widget = gtk_label_new (markup);
gtk_widget_set_margin_top (widget, 6);
gtk_label_set_use_markup (GTK_LABEL (widget), TRUE);
- gtk_misc_set_alignment (GTK_MISC (widget), 0.0, 0.5);
+ gtk_widget_set_halign (widget, GTK_ALIGN_START);
+ gtk_widget_set_valign (widget, GTK_ALIGN_CENTER);
gtk_box_pack_start (GTK_BOX (parent), widget, FALSE, FALSE, 0);
gtk_widget_show (widget);
g_free (markup);
widget = e_mail_config_auth_check_new (backend);
- gtk_widget_set_margin_left (widget, 12);
+ gtk_widget_set_margin_start (widget, 12);
gtk_box_pack_start (GTK_BOX (parent), widget, FALSE, FALSE, 0);
priv->auth_check = widget; /* do not reference */
gtk_widget_show (widget);
@@ -317,7 +323,6 @@ mail_config_eas_backend_check_complete (EMailConfigServiceBackend *backend)
EMailConfigServicePage *page;
CamelSettings *settings;
CamelNetworkSettings *network_settings;
- const gchar *email_address;
const gchar *username;
const gchar *hosturl;
@@ -335,8 +340,6 @@ mail_config_eas_backend_check_complete (EMailConfigServiceBackend *backend)
* introduce a backend extension in the mail transport source. */
settings = e_mail_config_service_backend_get_settings (backend);
- email_address = e_mail_config_service_page_get_email_address (page);
-
network_settings = CAMEL_NETWORK_SETTINGS (settings);
username = camel_network_settings_get_user (network_settings);
hosturl = camel_network_settings_get_host (network_settings);
@@ -367,7 +370,7 @@ mail_config_eas_backend_commit_changes (EMailConfigServiceBackend *backend)
* the backend is associated with. This method only applies to
* the Receiving Page. */
if (!E_IS_MAIL_CONFIG_RECEIVING_PAGE (page))
- return TRUE;
+ return;
/* This needs to come _after_ the page type check so we don't
* introduce a backend extension in the mail transport source. */
@@ -375,33 +378,35 @@ mail_config_eas_backend_commit_changes (EMailConfigServiceBackend *backend)
email_address = e_mail_config_service_page_get_email_address (page);
- GConfClient *client = gconf_client_get_default ();
- char *key;
-
- username = camel_network_settings_dup_user ((CamelNetworkSettings *)settings);
- g_strstrip (username);
+ if (email_address != NULL) {
+ GConfClient *client = gconf_client_get_default ();
+ char *key;
- if (username && username[0]) {
- key = g_strdup_printf ("/apps/activesyncd/accounts/%s/username", email_address);
- gconf_client_set_string (client, key, username, NULL);
- g_free (key);
- }
+ username = camel_network_settings_dup_user ((CamelNetworkSettings *)settings);
+ g_strstrip (username);
- g_free(username);
-
- hosturl = camel_network_settings_dup_host ((CamelEasSettings *)settings);
- g_strstrip (hosturl);
+ if (username && username[0]) {
+ key = g_strdup_printf ("/apps/activesyncd/accounts/%s/username", email_address);
+ gconf_client_set_string (client, key, username, NULL);
+ g_free (key);
+ }
- if (hosturl && hosturl[0]) {
- key = g_strdup_printf ("/apps/activesyncd/accounts/%s/serverUri", email_address);
- gconf_client_set_string (client, key, hosturl, NULL);
- g_free (key);
+ g_free(username);
- camel_eas_settings_set_account_uid ((CamelEasSettings *)settings, email_address);
- }
+ hosturl = camel_network_settings_dup_host ((CamelNetworkSettings *)settings);
+ g_strstrip (hosturl);
+
+ if (hosturl && hosturl[0]) {
+ key = g_strdup_printf ("/apps/activesyncd/accounts/%s/serverUri", email_address);
+ gconf_client_set_string (client, key, hosturl, NULL);
+ g_free (key);
- g_free(hosturl);
- g_object_unref (client);
+ camel_eas_settings_set_account_uid ((CamelEasSettings *)settings, email_address);
+ }
+
+ g_free(hosturl);
+ g_object_unref (client);
+ }
}
static void
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]