add -Wextra to our checks

ada-url · Feb 27, 2025 · a7c104c · a7c104c
1 parent 88f992e
commit a7c104c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/ubuntu_pedantic.yml b/.github/workflows/ubuntu_pedantic.yml
@@ -34,7 +34,7 @@ jobs:
         run: cmake -D ADA_TESTING=ON -DBUILD_SHARED_LIBS=${{matrix.shared}} -G Ninja -B build
         env:
           CXX: g++-12
-          CXXFLAGS: -Werror
+          CXXFLAGS: -Werror -Wextra
       - name: Build
         run: cmake --build build -j=4
       - name: Test

diff --git a/tests/wpt_urlpattern_tests.cpp b/tests/wpt_urlpattern_tests.cpp
@@ -146,9 +146,9 @@ std::variant<ada::url_pattern_init, ada::url_pattern_options> parse_init(
     auto key = field.key().value();
     std::string_view value;
     if (field.value().get_string(value)) {
-      bool value_true;
-      EXPECT_FALSE(field.value().get_bool().get(value_true));
-      return ada::url_pattern_options{.ignore_case = value_true};
+      bool ignore_case = false;
+      EXPECT_FALSE(field.value().get_bool().get(ignore_case));
+      return ada::url_pattern_options{.ignore_case = ignore_case};
     }
     if (key == "protocol") {
       init.protocol = std::string(value);