From: Steve Holme Date: Sat, 9 Mar 2013 18:50:41 +0000 (+0000) Subject: imap: Re-factored all perform based functions X-Git-Tag: upstream/7.37.1~1955 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=88264355710392d0536a1e11a5426fa6507a50f8;p=platform%2Fupstream%2Fcurl.git imap: Re-factored all perform based functions Standardised the naming of all perform based functions to be in the form imap_perform_something(). --- diff --git a/lib/imap.c b/lib/imap.c index 7d980e5..2d8fbbb 100644 --- a/lib/imap.c +++ b/lib/imap.c @@ -498,12 +498,12 @@ static void state(struct connectdata *conn, imapstate newstate) /*********************************************************************** * - * imap_state_capability() + * imap_perform_capability() * * Sends the CAPABILITY command in order to obtain a list of server side * supported capabilities. */ -static CURLcode imap_state_capability(struct connectdata *conn) +static CURLcode imap_perform_capability(struct connectdata *conn) { CURLcode result = CURLE_OK; struct imap_conn *imapc = &conn->proto.imapc; @@ -523,11 +523,11 @@ static CURLcode imap_state_capability(struct connectdata *conn) /*********************************************************************** * - * imap_state_starttls() + * imap_perform_starttls() * * Sends the STARTTLS command to start the upgrade to TLS. */ -static CURLcode imap_state_starttls(struct connectdata *conn) +static CURLcode imap_perform_starttls(struct connectdata *conn) { CURLcode result = CURLE_OK; @@ -542,11 +542,11 @@ static CURLcode imap_state_starttls(struct connectdata *conn) /*********************************************************************** * - * imap_state_upgrade_tls() + * imap_perform_upgrade_tls() * * Performs the upgrade to TLS. */ -static CURLcode imap_state_upgrade_tls(struct connectdata *conn) +static CURLcode imap_perform_upgrade_tls(struct connectdata *conn) { CURLcode result = CURLE_OK; struct imap_conn *imapc = &conn->proto.imapc; @@ -560,7 +560,7 @@ static CURLcode imap_state_upgrade_tls(struct connectdata *conn) if(imapc->ssldone) { imap_to_imaps(conn); - result = imap_state_capability(conn); + result = imap_perform_capability(conn); } } @@ -569,11 +569,11 @@ static CURLcode imap_state_upgrade_tls(struct connectdata *conn) /*********************************************************************** * - * imap_state_login() + * imap_perform_login() * * Sends a clear text LOGIN command to authenticate with. */ -static CURLcode imap_state_login(struct connectdata *conn) +static CURLcode imap_perform_login(struct connectdata *conn) { CURLcode result = CURLE_OK; char *user; @@ -606,7 +606,7 @@ static CURLcode imap_state_login(struct connectdata *conn) /*********************************************************************** * - * imap_authenticate() + * imap_perform_authenticate() * * Sends an AUTHENTICATE command allowing the client to login with the * appropriate SASL authentication mechanism. @@ -614,7 +614,7 @@ static CURLcode imap_state_login(struct connectdata *conn) * Additionally, the function will perform fallback to the LOGIN command * should a common mechanism not be available between the client and server. */ -static CURLcode imap_authenticate(struct connectdata *conn) +static CURLcode imap_perform_authenticate(struct connectdata *conn) { CURLcode result = CURLE_OK; struct imap_conn *imapc = &conn->proto.imapc; @@ -703,7 +703,7 @@ static CURLcode imap_authenticate(struct connectdata *conn) } else if(!imapc->login_disabled) /* Perform clear text authentication */ - result = imap_state_login(conn); + result = imap_perform_login(conn); else { /* Other mechanisms not supported */ infof(conn->data, "No known authentication mechanisms supported!\n"); @@ -715,11 +715,11 @@ static CURLcode imap_authenticate(struct connectdata *conn) /*********************************************************************** * - * imap_list() + * imap_perform_list() * * Sends a LIST command or an alternative custom request. */ -static CURLcode imap_list(struct connectdata *conn) +static CURLcode imap_perform_list(struct connectdata *conn) { CURLcode result = CURLE_OK; struct SessionHandle *data = conn->data; @@ -750,11 +750,11 @@ static CURLcode imap_list(struct connectdata *conn) /*********************************************************************** * - * imap_select() + * imap_perform_select() * * Sends a SELECT command to ask the server to change the selected mailbox. */ -static CURLcode imap_select(struct connectdata *conn) +static CURLcode imap_perform_select(struct connectdata *conn) { CURLcode result = CURLE_OK; struct SessionHandle *data = conn->data; @@ -790,11 +790,11 @@ static CURLcode imap_select(struct connectdata *conn) /*********************************************************************** * - * imap_fetch() + * imap_perform_fetch() * * Sends a FETCH command to initiate the download of a message. */ -static CURLcode imap_fetch(struct connectdata *conn) +static CURLcode imap_perform_fetch(struct connectdata *conn) { CURLcode result = CURLE_OK; struct IMAP *imap = conn->data->state.proto.imap; @@ -818,11 +818,11 @@ static CURLcode imap_fetch(struct connectdata *conn) /*********************************************************************** * - * imap_append() + * imap_perform_append() * * Sends an APPEND command to initiate the upload of a message. */ -static CURLcode imap_append(struct connectdata *conn) +static CURLcode imap_perform_append(struct connectdata *conn) { CURLcode result = CURLE_OK; struct IMAP *imap = conn->data->state.proto.imap; @@ -859,11 +859,11 @@ static CURLcode imap_append(struct connectdata *conn) /*********************************************************************** * - * imap_logout() + * imap_perform_logout() * * Performs the logout action prior to sclose() being called. */ -static CURLcode imap_logout(struct connectdata *conn) +static CURLcode imap_perform_logout(struct connectdata *conn) { CURLcode result = CURLE_OK; @@ -891,7 +891,7 @@ static CURLcode imap_state_servergreet_resp(struct connectdata *conn, result = CURLE_FTP_WEIRD_SERVER_REPLY; /* TODO: fix this code */ } else - result = imap_state_capability(conn); + result = imap_perform_capability(conn); return result; } @@ -973,20 +973,20 @@ static CURLcode imap_state_capability_resp(struct connectdata *conn, /* We don't have a SSL/TLS connection yet, but SSL is requested */ if(imapc->tls_supported) /* Switch to TLS connection now */ - result = imap_state_starttls(conn); + result = imap_perform_starttls(conn); else if(data->set.use_ssl == CURLUSESSL_TRY) /* Fallback and carry on with authentication */ - result = imap_authenticate(conn); + result = imap_perform_authenticate(conn); else { failf(data, "STARTTLS not supported."); result = CURLE_USE_SSL_FAILED; } } else - result = imap_authenticate(conn); + result = imap_perform_authenticate(conn); } else - result = imap_state_login(conn); + result = imap_perform_login(conn); return result; } @@ -1007,10 +1007,10 @@ static CURLcode imap_state_starttls_resp(struct connectdata *conn, result = CURLE_USE_SSL_FAILED; } else - result = imap_authenticate(conn); + result = imap_perform_authenticate(conn); } else - result = imap_state_upgrade_tls(conn); + result = imap_perform_upgrade_tls(conn); return result; } @@ -1427,9 +1427,9 @@ static CURLcode imap_state_select_resp(struct connectdata *conn, int imapcode, imapc->mailbox = strdup(imap->mailbox); if(imap->custom) - result = imap_list(conn); + result = imap_perform_list(conn); else - result = imap_fetch(conn); + result = imap_perform_fetch(conn); } } else { @@ -1606,7 +1606,7 @@ static CURLcode imap_statemach_act(struct connectdata *conn) /* Busy upgrading the connection; right now all I/O is SSL/TLS, not IMAP */ if(imapc->state == IMAP_UPGRADETLS) - return imap_state_upgrade_tls(conn); + return imap_perform_upgrade_tls(conn); /* Flush any data that needs to be sent */ if(pp->sendleft) @@ -1923,19 +1923,19 @@ static CURLcode imap_perform(struct connectdata *conn, bool *connected, /* Start the first command in the DO phase */ if(conn->data->set.upload) /* APPEND can be executed directly */ - result = imap_append(conn); + result = imap_perform_append(conn); else if(imap->custom && (selected || !imap->mailbox)) /* Custom command using the same mailbox or no mailbox */ - result = imap_list(conn); + result = imap_perform_list(conn); else if(!imap->custom && selected && imap->uid) /* FETCH from the same mailbox */ - result = imap_fetch(conn); + result = imap_perform_fetch(conn); else if(imap->mailbox && !selected && (imap->custom || imap->uid)) /* SELECT the mailbox */ - result = imap_select(conn); + result = imap_perform_select(conn); else /* LIST */ - result = imap_list(conn); + result = imap_perform_list(conn); if(result) return result; @@ -2008,7 +2008,7 @@ static CURLcode imap_disconnect(struct connectdata *conn, bool dead_connection) /* The IMAP session may or may not have been allocated/setup at this point! */ if(!dead_connection && imapc->pp.conn) - if(!imap_logout(conn)) + if(!imap_perform_logout(conn)) (void)imap_block_statemach(conn); /* ignore errors on LOGOUT */ /* Disconnect from the server */