Fix clang-tidy warnings

foonathan · Jan 7, 2025 · 2b4693d · 2b4693d
1 parent b08de16
commit 2b4693d
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 6 deletions.
diff --git a/include/lexy/_detail/config.hpp b/include/lexy/_detail/config.hpp
@@ -50,7 +50,7 @@ template <typename T>
 std::add_rvalue_reference_t<T> declval();
 
 template <typename T>
-constexpr void swap(T& lhs, T& rhs)
+constexpr void swap(T& lhs, T& rhs) noexcept
 {
     T tmp = LEXY_MOV(lhs);
     lhs   = LEXY_MOV(rhs);

diff --git a/include/lexy/_detail/memory_resource.hpp b/include/lexy/_detail/memory_resource.hpp
@@ -9,7 +9,7 @@
 #include <lexy/_detail/config.hpp>
 #include <new>
 
-#if 0
+#if 0 // NOLINT
 // Subset of the interface of std::pmr::memory_resource.
 class MemoryResource
 {

diff --git a/include/lexy/encoding.hpp b/include/lexy/encoding.hpp
@@ -12,7 +12,7 @@
 namespace lexy
 {
 /// The endianness used by an encoding.
-enum class encoding_endianness
+enum class encoding_endianness // NOLINT(performance-enum-size)
 {
     /// Little endian.
     little,

diff --git a/src/input/file.cpp b/src/input/file.cpp
@@ -6,7 +6,9 @@
 
 #include <cerrno>
 #include <cstdio>
+#include <lexy/_detail/assert.hpp>
 #include <lexy/_detail/buffer_builder.hpp>
+#include <lexy/input/buffer.hpp>
 
 #if defined(__unix__) || defined(__APPLE__)
 
@@ -68,7 +70,7 @@ lexy::file_error lexy::_detail::read_file(const char* path, file_callback cb, vo
         return get_file_error();
 
     auto off = ::lseek(fd, 0, SEEK_END);
-    if (off == static_cast<::off_t>(-1))
+    if (off == static_cast<decltype(off)>(-1))
         return lexy::file_error::os_error;
     auto size = static_cast<std::size_t>(off);
 

diff --git a/tests/doctest_main.cpp b/tests/doctest_main.cpp
@@ -1,8 +1,6 @@
 // Copyright (C) 2020-2024 Jonathan Müller and lexy contributors
 // SPDX-License-Identifier: BSL-1.0
 
-#include <lexy/_detail/config.hpp>
-
 #define DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN
 #include <doctest/doctest.h>
 
diff --git a/tests/lexy/detail/buffer_builder.cpp b/tests/lexy/detail/buffer_builder.cpp
@@ -3,6 +3,7 @@
 
 #include <lexy/_detail/buffer_builder.hpp>
 
+#include <cstring>
 #include <doctest/doctest.h>
 
 TEST_CASE("_detail::buffer_builder")

diff --git a/tests/lexy/detail/integer_sequence.cpp b/tests/lexy/detail/integer_sequence.cpp
@@ -4,6 +4,7 @@
 #include <lexy/_detail/integer_sequence.hpp>
 
 #include <doctest/doctest.h>
+#include <type_traits>
 
 TEST_CASE("_detail::integer_sequence")
 {