[evolution/wip/gsettings] Random little coding style cleanups.



commit 4a6232a228b7b45b339fc2bc596eb939a335b827
Author: Matthew Barnes <mbarnes redhat com>
Date:   Thu May 5 14:34:31 2011 -0400

    Random little coding style cleanups.

 mail/e-mail-local.c             |    3 +-
 mail/e-mail-reader-utils.c      |    3 +-
 mail/e-mail-session.c           |    1 -
 mail/em-filter-folder-element.c |   60 +++++++++++++++++++++-----------------
 4 files changed, 35 insertions(+), 32 deletions(-)
---
diff --git a/mail/e-mail-local.c b/mail/e-mail-local.c
index a61e9dc..a348813 100644
--- a/mail/e-mail-local.c
+++ b/mail/e-mail-local.c
@@ -107,8 +107,7 @@ e_mail_local_init (EMailSession *session,
 
 	camel_url_free (url);
 
-	g_object_ref (service);
-	local_store = CAMEL_STORE (service);
+	local_store = g_object_ref (service);
 
 	return;
 
diff --git a/mail/e-mail-reader-utils.c b/mail/e-mail-reader-utils.c
index cb3eef5..534a973 100644
--- a/mail/e-mail-reader-utils.c
+++ b/mail/e-mail-reader-utils.c
@@ -289,8 +289,7 @@ e_mail_reader_open_selected (EMailReader *reader)
 		GtkWidget *browser;
 
 		browser = e_mail_browser_new (backend);
-		e_mail_reader_set_folder (
-			E_MAIL_READER (browser), folder);
+		e_mail_reader_set_folder (E_MAIL_READER (browser), folder);
 		e_mail_reader_set_message (E_MAIL_READER (browser), uid);
 		copy_tree_state (reader, E_MAIL_READER (browser));
 		e_mail_reader_set_group_by_threads (
diff --git a/mail/e-mail-session.c b/mail/e-mail-session.c
index b69adf8..f8ca413 100644
--- a/mail/e-mail-session.c
+++ b/mail/e-mail-session.c
@@ -1238,7 +1238,6 @@ e_mail_session_uri_to_folder_sync (EMailSession *session,
 
 	if (folder != NULL) {
 		MailFolderCache *folder_cache;
-
 		folder_cache = e_mail_session_get_folder_cache (session);
 		mail_folder_cache_note_folder (folder_cache, folder);
 	}
diff --git a/mail/em-filter-folder-element.c b/mail/em-filter-folder-element.c
index 59a44f1..76828f7 100644
--- a/mail/em-filter-folder-element.c
+++ b/mail/em-filter-folder-element.c
@@ -225,7 +225,7 @@ em_filter_folder_element_set_uri (EMFilterFolderElement *element,
 static gboolean
 validate (EFilterElement *fe, EAlert **alert)
 {
-	EMFilterFolderElement *ff = (EMFilterFolderElement *)fe;
+	EMFilterFolderElement *ff = (EMFilterFolderElement *) fe;
 
 	g_warn_if_fail (alert == NULL || *alert == NULL);
 
@@ -244,25 +244,25 @@ folder_eq (EFilterElement *fe, EFilterElement *cm)
 {
 	return E_FILTER_ELEMENT_CLASS (
 		em_filter_folder_element_parent_class)->eq (fe, cm) &&
-		strcmp (((EMFilterFolderElement *)fe)->priv->uri,
-		((EMFilterFolderElement *)cm)->priv->uri)== 0;
+		strcmp (((EMFilterFolderElement *) fe)->priv->uri,
+		((EMFilterFolderElement *) cm)->priv->uri)== 0;
 }
 
 static xmlNodePtr
 xml_encode (EFilterElement *fe)
 {
 	xmlNodePtr value, work;
-	EMFilterFolderElement *ff = (EMFilterFolderElement *)fe;
+	EMFilterFolderElement *ff = (EMFilterFolderElement *) fe;
 
-	value = xmlNewNode(NULL, (const guchar *)"value");
-	xmlSetProp(value, (const guchar *)"name", (guchar *)fe->name);
+	value = xmlNewNode(NULL, (xmlChar *) "value");
+	xmlSetProp(value, (xmlChar *) "name", (xmlChar *) fe->name);
 	if (ff->store_camel_uri)
-		xmlSetProp(value, (const guchar *)"type", (const guchar *)"folder-curi");
+		xmlSetProp(value, (xmlChar *) "type", (xmlChar *) "folder-curi");
 	else
-		xmlSetProp(value, (const guchar *)"type", (const guchar *)"folder");
+		xmlSetProp(value, (xmlChar *) "type", (xmlChar *) "folder");
 
-	work = xmlNewChild(value, NULL, (const guchar *)"folder", NULL);
-	xmlSetProp(work, (const guchar *)"uri", (const guchar *)ff->priv->uri);
+	work = xmlNewChild(value, NULL, (xmlChar *) "folder", NULL);
+	xmlSetProp(work, (xmlChar *) "uri", (xmlChar *) ff->priv->uri);
 
 	return value;
 }
@@ -270,16 +270,16 @@ xml_encode (EFilterElement *fe)
 static gint
 xml_decode (EFilterElement *fe, xmlNodePtr node)
 {
-	EMFilterFolderElement *ff = (EMFilterFolderElement *)fe;
+	EMFilterFolderElement *ff = (EMFilterFolderElement *) fe;
 	xmlNodePtr n;
 	xmlChar *type;
 
 	xmlFree (fe->name);
-	fe->name = (gchar *)xmlGetProp(node, (const guchar *)"name");
+	fe->name = (gchar *) xmlGetProp(node, (xmlChar *) "name");
 
-	type = xmlGetProp (node, (const guchar *)"type");
+	type = xmlGetProp (node, (xmlChar *) "type");
 	if (type) {
-		ff->store_camel_uri = g_str_equal ((const gchar *)type, "folder-curi");
+		ff->store_camel_uri = g_str_equal ((const gchar *) type, "folder-curi");
 		xmlFree (type);
 	} else {
 		ff->store_camel_uri = FALSE;
@@ -287,10 +287,10 @@ xml_decode (EFilterElement *fe, xmlNodePtr node)
 
 	n = node->children;
 	while (n) {
-		if (!strcmp((gchar *)n->name, "folder")) {
+		if (!strcmp((gchar *) n->name, "folder")) {
 			gchar *uri;
 
-			uri = (gchar *)xmlGetProp(n, (const guchar *)"uri");
+			uri = (gchar *) xmlGetProp(n, (xmlChar *) "uri");
 			g_free (ff->priv->uri);
 			ff->priv->uri = g_strdup (uri);
 			xmlFree (uri);
@@ -303,7 +303,8 @@ xml_decode (EFilterElement *fe, xmlNodePtr node)
 }
 
 static void
-folder_selected (EMFolderSelectionButton *button, EMFilterFolderElement *ff)
+folder_selected (EMFolderSelectionButton *button,
+                 EMFilterFolderElement *ff)
 {
 	GtkWidget *toplevel;
 	const gchar *uri;
@@ -323,7 +324,7 @@ folder_selected (EMFolderSelectionButton *button, EMFilterFolderElement *ff)
 static GtkWidget *
 get_widget (EFilterElement *fe)
 {
-	EMFilterFolderElement *ff = (EMFilterFolderElement *)fe;
+	EMFilterFolderElement *ff = (EMFilterFolderElement *) fe;
 	EMailSession *session;
 	GtkWidget *button;
 	gchar *uri;
@@ -345,7 +346,10 @@ get_widget (EFilterElement *fe)
 		g_free (uri);
 
 	gtk_widget_show (button);
-	g_signal_connect(button, "selected", G_CALLBACK(folder_selected), ff);
+
+	g_signal_connect (
+		button, "selected",
+		G_CALLBACK (folder_selected), ff);
 
 	return button;
 }
@@ -359,7 +363,7 @@ build_code (EFilterElement *fe, GString *out, EFilterPart *ff)
 static void
 format_sexp (EFilterElement *fe, GString *out)
 {
-	EMFilterFolderElement *ff = (EMFilterFolderElement *)fe;
+	EMFilterFolderElement *ff = (EMFilterFolderElement *) fe;
 
 	e_sexp_encode_string (out, ff->priv->uri);
 }
@@ -368,11 +372,13 @@ static void
 emff_copy_value (EFilterElement *de, EFilterElement *se)
 {
 	if (EM_IS_FILTER_FOLDER_ELEMENT (se)) {
-		((EMFilterFolderElement *)de)->store_camel_uri =
-			((EMFilterFolderElement *)se)->store_camel_uri;
-		em_filter_folder_element_set_uri ((
-			EMFilterFolderElement *) de,
-			((EMFilterFolderElement *) se)->priv->uri);
-	} else
-		E_FILTER_ELEMENT_CLASS (em_filter_folder_element_parent_class)->copy_value (de, se);
+		((EMFilterFolderElement *) de)->store_camel_uri =
+			((EMFilterFolderElement *) se)->store_camel_uri;
+		em_filter_folder_element_set_uri (
+			EM_FILTER_FOLDER_ELEMENT (de),
+			EM_FILTER_FOLDER_ELEMENT (se)->priv->uri);
+	} else {
+		E_FILTER_ELEMENT_CLASS (
+		em_filter_folder_element_parent_class)->copy_value (de, se);
+	}
 }



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]