imap: Tidied up the LIST response function

Reworked comments as they referenced custom commands, removed
unnecessary state change on failure and setting of result code on
success.

diff --git a/lib/imap.c b/lib/imap.c
index 90a2eb3284ef8e61212c6aaad5b62e1d1e4f3dae..3130e1abb5eed45b76e863ca7611bc66e89eace3 100644 (file)
--- a/lib/imap.c
+++ b/lib/imap.c
@@ -1323,23 +1323,16 @@ static CURLcode imap_state_list_resp(struct connectdata *conn, int imapcode,
   (void)instate; /* No use for this yet */
 
   if(imapcode == '*') {
-    /* The client which asked for this custom command should know best
-       how to cope with the result, just send it as body.
-       Add back the LF character temporarily while saving. */
+    /* Temporarily add the LF character back and send as body to the client */
     line[len] = '\n';
     result = Curl_client_write(conn, CLIENTWRITE_BODY, line, len + 1);
     line[len] = '\0';
   }
-  else {
-    /* Final response. Stop and return the final status. */
-    if(imapcode != 'O')
-      result = CURLE_QUOTE_ERROR; /* TODO: Fix error code */
-    else
-      result = CURLE_OK;
-
+  else if(imapcode != 'O')
+    result = CURLE_QUOTE_ERROR; /* TODO: Fix error code */
+  else
     /* End of DO phase */
     state(conn, IMAP_STOP);
-  }
 
   return result;
 }