fix warnings

anonrig · anonrig · commit 380d57e3ad5f · 2024-12-12T13:04:32.000-05:00
diff --git a/include/ada/url_pattern-inl.h b/include/ada/url_pattern-inl.h
@@ -313,7 +313,7 @@ inline std::string_view constructor_string_parser::make_component_string() {
   const auto end_index = token.index;
   // Return the code point substring from component start input index to end
   // index within parser&rsquo;s input.
-  return input.substr(component_start_input_index, end_index);
+  return std::string_view(input).substr(component_start_input_index, end_index);
 }
 
 inline bool constructor_string_parser::is_an_identity_terminator() {
@@ -360,9 +360,9 @@ inline void Tokenizer::get_next_code_point() {
   next_index++;
 }
 
-inline void Tokenizer::seek_and_get_next_code_point(size_t index) {
+inline void Tokenizer::seek_and_get_next_code_point(size_t new_index) {
   // Set tokenizer&rsquo;s next index to index.
-  next_index = index;
+  next_index = new_index;
   // Run get the next code point given tokenizer.
   get_next_code_point();
 }
@@ -405,13 +405,13 @@ Tokenizer::process_tokenizing_error(size_t next_position,
 }
 
 // @see https://urlpattern.spec.whatwg.org/#is-a-valid-name-code-point
-inline bool Tokenizer::is_valid_name_code_point(char code_point, bool first) {
+inline bool Tokenizer::is_valid_name_code_point(char cp, bool first) {
   // If first is true return the result of checking if code point is contained
   // in the IdentifierStart set of code points. Otherwise return the result of
   // checking if code point is contained in the IdentifierPart set of code
   // points.
   // TODO: Implement this
-  (void)code_point;
+  (void)cp;
   (void)first;
   return true;
 }
diff --git a/include/ada/url_pattern.h b/include/ada/url_pattern.h
@@ -148,9 +148,9 @@ struct url_pattern_part {
 struct url_pattern_compile_component_options {
   url_pattern_compile_component_options() = default;
   explicit url_pattern_compile_component_options(
-      std::optional<char> delimiter = std::nullopt,
-      std::optional<char> prefix = std::nullopt)
-      : delimiter(delimiter), prefix(prefix){};
+      std::optional<char> new_delimiter = std::nullopt,
+      std::optional<char> new_prefix = std::nullopt)
+      : delimiter(new_delimiter), prefix(new_prefix){};
 
   // @see https://urlpattern.spec.whatwg.org/#options-delimiter-code-point
   std::optional<char> delimiter{};
@@ -338,7 +338,7 @@ struct Token {
 
 // @see https://urlpattern.spec.whatwg.org/#tokenizer
 class Tokenizer {
-public:
+ public:
   explicit Tokenizer(std::string_view input, token_policy policy)
       : input(input), policy(policy) {}
 
