From e94fbcabd77d51dd3a01e0d3dec3646abaa5742e Mon Sep 17 00:00:00 2001
From: Steve Holme <steve_holme@hotmail.com>
Date: Sat, 2 Mar 2013 20:40:38 +0000
Subject: imap: Simplified the imap_state_append_resp() function

Introduced the result code variable to simplify the state changes and
remove the hard returns.
---
 lib/imap.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'lib')

diff --git a/lib/imap.c b/lib/imap.c
index 263296142..c61231b2a 100644
--- a/lib/imap.c
+++ b/lib/imap.c
@@ -1391,23 +1391,23 @@ static CURLcode imap_state_append_resp(struct connectdata *conn,
                                        int imapcode,
                                        imapstate instate)
 {
+  CURLcode result = CURLE_OK;
   struct SessionHandle *data = conn->data;
 
   (void)instate; /* No use for this yet */
 
   if(imapcode != '+') {
-    state(conn, IMAP_STOP);
-    return CURLE_UPLOAD_FAILED;
+    result = CURLE_UPLOAD_FAILED;
   }
   else {
     Curl_pgrsSetUploadSize(data, data->set.infilesize);
     Curl_setup_transfer(conn, -1, -1, FALSE, NULL, /* No download */
                         FIRSTSOCKET, NULL);
-
-    /* Stop now and let the core go from DO to PERFORM phase */
-    state(conn, IMAP_STOP);
-    return CURLE_OK;
   }
+
+  state(conn, IMAP_STOP);
+
+  return result;
 }
 
 /* For final APPEND responses performed after the upload */
-- 
cgit v1.2.3