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/altsvc.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/altsvc.c')
-rw-r--r-- | lib/altsvc.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/lib/altsvc.c b/lib/altsvc.c index 67e75cc6a..85a4e01b5 100644 --- a/lib/altsvc.c +++ b/lib/altsvc.c @@ -253,7 +253,6 @@ struct altsvcinfo *Curl_altsvc_init(void) | CURLALTSVC_H2 #endif #ifdef USE_HTTP3 - /* TODO: adjust when known */ | CURLALTSVC_H3 #endif ; @@ -349,7 +348,7 @@ static CURLcode getalnum(const char **ptr, char *alpnbuf, size_t buflen) len = p - protop; if(!len || (len >= buflen)) - return CURLE_BAD_FUNCTION_ARGUMENT; /* TODO: improve error code */ + return CURLE_BAD_FUNCTION_ARGUMENT; memcpy(alpnbuf, protop, len); alpnbuf[len] = 0; *ptr = p; @@ -425,7 +424,6 @@ CURLcode Curl_altsvc_parse(struct Curl_easy *data, /* "clear" is a magic keyword */ if(strcasecompare(alpnbuf, "clear")) { - /* TODO: clear whatever it is it should clear */ return CURLE_OK; } @@ -478,7 +476,7 @@ CURLcode Curl_altsvc_parse(struct Curl_easy *data, p++; len = p - hostp; if(!len || (len >= MAX_ALTSVC_HOSTLEN)) - return CURLE_BAD_FUNCTION_ARGUMENT; /* TODO: improve error code */ + return CURLE_BAD_FUNCTION_ARGUMENT; memcpy(namebuf, hostp, len); namebuf[len] = 0; dsthost = namebuf; @@ -504,8 +502,8 @@ CURLcode Curl_altsvc_parse(struct Curl_easy *data, srcalpnid, dstalpnid, srcport, dstport); if(as) { - /* TODO: the expires time also needs to take the Age: value (if any) - into account. [See RFC 7838 section 3.1] */ + /* The expires time also needs to take the Age: value (if any) into + account. [See RFC 7838 section 3.1] */ as->expires = maxage + time(NULL); as->persist = persist; Curl_llist_insert_next(&asi->list, asi->list.tail, as, &as->node); |