aboutsummaryrefslogtreecommitdiffstats
path: root/capplet
diff options
context:
space:
mode:
authorFederico Mena Quintero <federico@novell.com>2010-03-17 03:08:33 +0800
committerFederico Mena Quintero <federico@novell.com>2010-03-18 06:08:15 +0800
commit5cedd8e7692a0b36267281b0465f50b3e80bd5fc (patch)
treeb5a91174b347e7703e5ab2e80c2eef79eec6a90a /capplet
parent65b0abce67e1bc564c4c74c617fd239883e5adda (diff)
downloadgsoc2013-evolution-5cedd8e7692a0b36267281b0465f50b3e80bd5fc.tar.gz
gsoc2013-evolution-5cedd8e7692a0b36267281b0465f50b3e80bd5fc.tar.zst
gsoc2013-evolution-5cedd8e7692a0b36267281b0465f50b3e80bd5fc.zip
Return a success code when parsing a message
The child functions of parse_message() do not yet return error values, but this will make it easier to do something with them in the future if they get added. Signed-off-by: Federico Mena Quintero <federico@novell.com>
Diffstat (limited to 'capplet')
-rw-r--r--capplet/settings/mail-guess-servers.c10
1 files changed, 6 insertions, 4 deletions
diff --git a/capplet/settings/mail-guess-servers.c b/capplet/settings/mail-guess-servers.c
index 891bdb1585..e5324e9e5a 100644
--- a/capplet/settings/mail-guess-servers.c
+++ b/capplet/settings/mail-guess-servers.c
@@ -159,7 +159,7 @@ handle_outgoing (xmlNodePtr head, EmailProvider *provider)
}
}
-static void
+static gboolean
parse_message (const char *msg, int length, EmailProvider *provider)
{
xmlDocPtr doc;
@@ -177,7 +177,7 @@ parse_message (const char *msg, int length, EmailProvider *provider)
if (!node) {
g_warning ("Incorrect data: ClientConfig not found ... Quitting\n");
- return;
+ return FALSE;
}
node = node->children;
@@ -190,7 +190,7 @@ parse_message (const char *msg, int length, EmailProvider *provider)
if (!node) {
g_warning ("Incorrect data: ClientConfig not found ... Quitting\n");
- return;
+ return FALSE;
}
top = node;
@@ -208,12 +208,14 @@ parse_message (const char *msg, int length, EmailProvider *provider)
}
xmlFreeDoc(doc);
+
+ return TRUE;
}
static gboolean
parse_soup_message (SoupMessage *msg, EmailProvider *provider)
{
- parse_message (msg->response_body->data, msg->response_body->length, provider);
+ return parse_message (msg->response_body->data, msg->response_body->length, provider);
}
gboolean