Note that GCC's va_list seems to be implemented as some type with an array size
of 1 (__va_list_tag[1]), which is expected to implicitly decay to a pointer
when passed as a parameter. This unfortunately necessitates NOLINTBEGIN/END
where it's used to silence clang-tidy.
* clang-tidy: use bool literals
Found with modernize-use-bool-literals
Signed-off-by: Rosen Penev <rosenp@gmail.com>
* clang-tidy: replace std::bind with lambdas
Found with modernize-avoid-bind
Signed-off-by: Rosen Penev <rosenp@gmail.com>
* clang-tidy: use data() instead of pointer stuff
Found with readability-container-data-pointe
Signed-off-by: Rosen Penev <rosenp@gmail.com>
* clang-tidy: use empty()
Found with readability-container-size-empty
Signed-off-by: Rosen Penev <rosenp@gmail.com>
* clang-tidy: remove static in anon namespace
Found with readability-static-definition-in-anonymous-namespace
Signed-off-by: Rosen Penev <rosenp@gmail.com>
* clang-tidy: remove const return
Found with readability-const-return-type
Signed-off-by: Rosen Penev <rosenp@gmail.com>
Signed-off-by: Rosen Penev <rosenp@gmail.com>
