fix: don't use alternative operators

Author: tearfur

include/small/detail/container/lookup_table_view.hpp

@@ -440,7 +440,7 @@ namespace small {
                 [[nodiscard]] constexpr bool
                 is_at_step_size() const {
                     return multibyte_index_ % step_size == step_size - 1
-                           && not is_end();
+                           && !is_end();
                 }
 
                 /// \brief Check if this is the end iterator
@@ -1646,7 +1646,7 @@ namespace small {
                     it = first;
                     step = search_range / 2;
                     std::advance(it, step);
-                    if (not it.contains_codepoint_information()) {
+                    if (!it.contains_codepoint_information()) {
                         // We can go a few indexes ahead or behind to fix this
                         const size_type indexes_behind = (step_size - 1)
                                                          - (it.multibyte_index()

include/small/detail/iterator/codepoint_iterator.hpp

@@ -91,7 +91,7 @@ namespace small {
                 /// value This should work in O(1) time because we have a hint
                 /// to the byte index
                 operator typename String::wide_value_type() const noexcept {
-                    if constexpr (not is_utf32_v<value_type>) {
+                    if constexpr (!is_utf32_v<value_type>) {
                         // Get reference to element and convert from utf8 to
                         // wide type
                         wide_value_type v;
@@ -126,7 +126,7 @@ namespace small {
                 operator<<(
                     std::ostream &os,
                     const codepoint_reference_impl &impl) {
-                    if constexpr (not is_utf32_v<value_type>) {
+                    if constexpr (!is_utf32_v<value_type>) {
                         if (impl.size() == 1) {
                             os << static_cast<char>(
                                 impl.container_->operator[](impl.byte_index_));

test/unit/small_string_make.cpp

@@ -465,7 +465,7 @@ TEST_CASE("String Rule of 5") {
                 REQUIRE(dv[0] == '\0');
             }
 
-            if constexpr (not is_windows()) {
+            if constexpr (!is_windows()) {
                 SECTION("Move and set alloc") {
                     std::allocator<int> alloc;
                     // There's no safe way to do that on MSVC :O
@@ -496,7 +496,7 @@ TEST_CASE("String Rule of 5") {
     }
 
     SECTION("Assignment Operator") {
-        if constexpr (not is_windows()) {
+        if constexpr (!is_windows()) {
             SECTION("String") {
                 string dv = u8"1😀2😀3😀";
                 string d;

test/unit/small_string_modify_algorithms.cpp

@@ -304,7 +304,7 @@ TEST_CASE("String Modifying Algorithms") {
                 }
             }
 
-            if constexpr (not is_windows()) {
+            if constexpr (!is_windows()) {
                 SECTION("UTF32 rhs") {
                     string a = u8"😐🙂😀🙂😀😐";
                     SECTION("Starts not-empty") {