aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorSteve Holme <steve_holme@hotmail.com>2013-02-25 15:53:24 +0000
committerSteve Holme <steve_holme@hotmail.com>2013-02-25 15:53:24 +0000
commit015ceb47062d7454defb48333c65968ff47f2e66 (patch)
tree4ca82f69dd2f305094363372305aefcc29bd84a0 /lib
parente63c33d88d8c6fda91b11afe44e18fdb05b94c17 (diff)
imap: Minor tidy up of comments in imap_parse_url_path()
Tidy up of comments before next round of imap changes.
Diffstat (limited to 'lib')
-rw-r--r--lib/imap.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/lib/imap.c b/lib/imap.c
index 8f1112d36..adde5423c 100644
--- a/lib/imap.c
+++ b/lib/imap.c
@@ -1720,7 +1720,7 @@ static CURLcode imap_parse_url_path(struct connectdata *conn)
char *value;
size_t valuelen;
- /* Decode the parameter name */
+ /* Find the length of the name parameter */
begin = ++ptr;
while(*ptr && *ptr != '=')
ptr++;
@@ -1728,16 +1728,17 @@ static CURLcode imap_parse_url_path(struct connectdata *conn)
if(!*ptr)
return CURLE_URL_MALFORMAT;
- /* Decode the parameter name */
+ /* Decode the name parameter */
result = Curl_urldecode(data, begin, ptr - begin, &name, NULL, TRUE);
if(result)
return result;
+ /* Find the length of the value parameter */
begin = ++ptr;
while(imap_is_bchar(*ptr))
ptr++;
- /* Decode the parameter value */
+ /* Decode the value parameter */
result = Curl_urldecode(data, begin, ptr - begin, &value, &valuelen, TRUE);
if(result) {
Curl_safefree(name);
@@ -1746,9 +1747,10 @@ static CURLcode imap_parse_url_path(struct connectdata *conn)
DEBUGF(infof(conn->data, "IMAP URL parameter '%s' = '%s'\n", name, value));
- /* Process known parameters (UIDVALIDITY, UID and SECTION) and create a
- virtual URL level, as they should be followed by a slash, which needs
- to be stripped off. Unknown parameters trigger a URL_MALFORMAT error */
+ /* Process the known hierarchical parameters (UIDVALIDITY, UID and SECTION)
+ stripping of the trailing slash character if it is present.
+
+ Note: Unknown parameters trigger a URL_MALFORMAT error. */
if(Curl_raw_equal(name, "UIDVALIDITY") && !imap->uidvalidity) {
if(valuelen > 0 && value[valuelen - 1] == '/')
value[valuelen - 1] = '\0';