Merge pull request #1717 from contour-terminal/improvement/small_tweaks

Small changes to validate key mapping documentation and no assert when compiling with CMAKE_BUILD_TYPE None

.github/actions/spelling/allow/uncategorized.txt

@@ -32,6 +32,7 @@ DATAROOTDIR
 DBus
 DCCACHE
 DCMAKE
+DNDEBUG
 DCODE
 DCONTOUR
 DEFGHIJKL

CMakeLists.txt

@@ -44,6 +44,12 @@ if(DEFINED MSVC)
     add_compile_options(/nologo)
 endif()
 
+
+# CMAKE_BUILD_TYPE check for None
+if(CMAKE_BUILD_TYPE STREQUAL "None")
+    add_compile_definitions(-DNDEBUG)
+endif()
+
 if (CONTOUR_BUILD_WITH_MIMALLOC)
     add_definitions(-DCONTOUR_BUILD_WITH_MIMALLOC)
 endif()

src/contour/Actions.h

@@ -282,7 +282,11 @@ namespace documentation
     constexpr inline std::string_view SwitchToTabRight { "Switch to tab to the right" };
 } // namespace documentation
 
-inline auto getDocumentation()
+#if defined(__clang__) && __clang_major__ >= 19
+constexpr
+#endif
+    inline auto
+    getDocumentation()
 {
     return std::array {
         std::tuple { Action { CancelSelection {} }, documentation::CancelSelection },
@@ -348,6 +352,10 @@ inline auto getDocumentation()
     };
 }
 
+#if defined(__clang__) && __clang_major__ >= 19
+static_assert(getDocumentation().size() == std::variant_size_v<Action>);
+#endif
+
 } // namespace contour::actions
 
 // {{{ fmtlib custom formatters