Fix: Various clang-tidy warnings

Else after return, usage after move, redundant casts, unused parameters.
spnda · Apr 28, 2024 · 4272301 · 4272301
1 parent e38682b
commit 4272301
Show file tree

Hide file tree

Showing 2 changed files with 109 additions and 113 deletions.
diff --git a/include/fastgltf/util.hpp b/include/fastgltf/util.hpp
@@ -28,6 +28,7 @@
 
 #include <array>
 #include <cmath>
+#include <cstddef>
 #include <cstdint>
 #include <limits>
 #include <memory>
@@ -110,7 +111,7 @@ namespace fastgltf {
 #if FASTGLTF_HAS_CONCEPTS
     requires std::is_enum_v<T>
 #endif
-    [[nodiscard]] constexpr std::underlying_type_t<T> to_underlying(T t) noexcept {
+    [[nodiscard, msvc::intrinsic]] constexpr std::underlying_type_t<T> to_underlying(T t) noexcept {
 #if !FASTGLTF_HAS_CONCEPTS
         static_assert(std::is_enum_v<T>, "to_underlying only works with enum types.");
 #endif