Widen pattern type from &str to S: AsRef<str>

crates/jlabel-question/src/lib.rs

@@ -166,7 +166,7 @@ where
     Self: Sized,
 {
     /// Parses question patterns in string, and if succeeds, returns the parsed question.
-    fn parse(patterns: &[&str]) -> Result<Self, ParseError>;
+    fn parse<S: AsRef<str>>(patterns: &[S]) -> Result<Self, ParseError>;
 
     /// Checks if the full-context label matches the question.
     ///
@@ -193,12 +193,12 @@ pub enum AllQuestion {
 }
 
 impl QuestionMatcher for AllQuestion {
-    fn parse(patterns: &[&str]) -> Result<Self, ParseError> {
+    fn parse<S: AsRef<str>>(patterns: &[S]) -> Result<Self, ParseError> {
         let mut position = None;
         let mut ranges = Vec::with_capacity(patterns.len());
 
         for pattern in patterns {
-            let (pos, range) = estimate_position(pattern)?;
+            let (pos, range) = estimate_position(pattern.as_ref())?;
 
             if let Some(position) = position {
                 if pos != position {

crates/jlabel-question/src/regex.rs

@@ -36,7 +36,7 @@ impl RegexQuestion {
 }
 
 impl QuestionMatcher for RegexQuestion {
-    fn parse(patterns: &[&str]) -> Result<Self, ParseError> {
+    fn parse<S: AsRef<str>>(patterns: &[S]) -> Result<Self, ParseError> {
         let regex = Regex::builder()
             .build_from_hir(&Hir::alternation(
                 patterns.iter().map(Self::parse_wildcard).collect(),

crates/jlabel-question/src/tests.rs

@@ -69,7 +69,7 @@ fn parse_question_err() {
     use ParseError::*;
     use PositionError::*;
 
-    assert_eq!(AllQuestion::parse(&[]), Err(Empty));
+    assert_eq!(AllQuestion::parse::<&str>(&[]), Err(Empty));
     assert_eq!(
         AllQuestion::parse(&["*/A:*"]),
         Err(InvalidPosition(EmptyRange))