From 5cedd8e7692a0b36267281b0465f50b3e80bd5fc Mon Sep 17 00:00:00 2001 From: Federico Mena Quintero Date: Tue, 16 Mar 2010 13:08:33 -0600 Subject: 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 --- capplet/settings/mail-guess-servers.c | 10 ++++++---- 1 file 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 -- cgit v1.2.3