diff options
author | Daniel Stenberg <daniel@haxx.se> | 2019-05-14 16:36:15 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2019-05-16 09:16:56 +0200 |
commit | 8ece8177f1e8ed8c76a7a6f3c90a23c5982b4641 (patch) | |
tree | 75949f9def8fd7feb86aa299360f2445a9c7a34e /lib/imap.c | |
parent | f506ce099f1ba0f659ff574d2ab09cfb18e8a203 (diff) | |
download | curl-8ece8177f1e8ed8c76a7a6f3c90a23c5982b4641.tar.gz |
cleanup: remove FIXME and TODO comments
They serve very little purpose and mostly just add noise. Most of them
have been around for a very long time. I read them all before removing
or rephrasing them.
Ref: #3876
Closes #3883
Diffstat (limited to 'lib/imap.c')
-rw-r--r-- | lib/imap.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/lib/imap.c b/lib/imap.c index ad20d9a4f..bdcc69c67 100644 --- a/lib/imap.c +++ b/lib/imap.c @@ -1043,7 +1043,7 @@ static CURLcode imap_state_listsearch_resp(struct connectdata *conn, line[len] = '\0'; } else if(imapcode != IMAP_RESP_OK) - result = CURLE_QUOTE_ERROR; /* TODO: Fix error code */ + result = CURLE_QUOTE_ERROR; else /* End of DO phase */ state(conn, IMAP_STOP); @@ -1115,7 +1115,7 @@ static CURLcode imap_state_fetch_resp(struct connectdata *conn, int imapcode, if(imapcode != '*') { Curl_pgrsSetDownloadSize(data, -1); state(conn, IMAP_STOP); - return CURLE_REMOTE_FILE_NOT_FOUND; /* TODO: Fix error code */ + return CURLE_REMOTE_FILE_NOT_FOUND; } /* Something like this is received "* 1 FETCH (BODY[TEXT] {2021}\r" so parse @@ -1492,12 +1492,7 @@ static CURLcode imap_done(struct connectdata *conn, CURLcode status, state(conn, IMAP_APPEND_FINAL); } - /* Run the state-machine - - TODO: when the multi interface is used, this _really_ should be using - the imap_multi_statemach function but we have no general support for - non-blocking DONE operations! - */ + /* Run the state-machine */ if(!result) result = imap_block_statemach(conn, FALSE); } |