diff options
author | Daniel Stenberg <daniel@haxx.se> | 2019-05-14 16:36:15 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2019-05-14 16:36:15 +0200 |
commit | 7f579d928e1beddce2c26f17debfb89010ecd513 (patch) | |
tree | 22345d37dc0d71d420234eae8848507ca700a215 /include/curl/typecheck-gcc.h | |
parent | be9fd703e0e919c7705c10fae6318afb2fda9f19 (diff) | |
download | curl-bagder/fixme-todo.tar.gz |
cleanup: remove FIXME and TODO commentsbagder/fixme-todo
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
Diffstat (limited to 'include/curl/typecheck-gcc.h')
-rw-r--r-- | include/curl/typecheck-gcc.h | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/include/curl/typecheck-gcc.h b/include/curl/typecheck-gcc.h index 8018ea37f..2d1de4d43 100644 --- a/include/curl/typecheck-gcc.h +++ b/include/curl/typecheck-gcc.h @@ -113,7 +113,6 @@ __extension__ ({ \ }) /* wraps curl_easy_getinfo() with typechecking */ -/* FIXME: don't allow const pointers */ #define curl_easy_getinfo(handle, info, arg) \ __extension__ ({ \ __typeof__(info) _curl_info = info; \ @@ -146,9 +145,8 @@ __extension__ ({ \ curl_easy_getinfo(handle, _curl_info, arg); \ }) -/* TODO: typechecking for curl_share_setopt() and curl_multi_setopt(), - * for now just make sure that the functions are called with three - * arguments +/* + * For now, just make sure that the functions are called with three arguments */ #define curl_share_setopt(share,opt,param) curl_share_setopt(share,opt,param) #define curl_multi_setopt(handle,opt,param) curl_multi_setopt(handle,opt,param) @@ -506,10 +504,6 @@ _CURL_WARNING(_curl_easy_getinfo_err_curl_off_t, _curl_is_arr((expr), char) || \ _curl_is_arr((expr), unsigned char)) -/* FIXME: the whole callback checking is messy... - * The idea is to tolerate char vs. void and const vs. not const - * pointers in arguments at least - */ /* helper: __builtin_types_compatible_p distinguishes between functions and * function pointers, hide it */ #define _curl_callback_compatible(func, type) \ |