diff options
author | Federico Mena Quintero <federico@novell.com> | 2010-03-17 03:08:33 +0800 |
---|---|---|
committer | Federico Mena Quintero <federico@novell.com> | 2010-03-18 06:08:15 +0800 |
commit | 5cedd8e7692a0b36267281b0465f50b3e80bd5fc (patch) | |
tree | b5a91174b347e7703e5ab2e80c2eef79eec6a90a /capplet/settings | |
parent | 65b0abce67e1bc564c4c74c617fd239883e5adda (diff) | |
download | gsoc2013-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/settings')
-rw-r--r-- | capplet/settings/mail-guess-servers.c | 10 |
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 |