Use is_no_match() helper for better maintainability

Address code review feedback to use the is_no_match() helper method
instead of pattern matching both NoMatch variants explicitly.

Co-authored-by: tomtau <[email protected]>

Author: Copilot

derive/src/from_pest/field.rs

@@ -98,8 +98,7 @@ impl ConversionStrategy {
                     // Try to parse using FromPest first
                     match ::from_pest::FromPest::from_pest(inner) {
                         Ok(value) => value,
-                        Err(::from_pest::ConversionError::NoMatch) |
-                        Err(::from_pest::ConversionError::NoMatchWithInfo { .. }) => #default_expr,
+                        Err(ref e) if e.is_no_match() => #default_expr,
                         Err(e) => return Err(e),
                     }
                 }}

src/lib.rs

@@ -141,7 +141,7 @@ impl<'pest, Rule: RuleType, T: FromPest<'pest, Rule = Rule>> FromPest<'pest> for
     type FatalError = T::FatalError;
     fn from_pest(pest: &mut Pairs<'pest, Rule>) -> Result<Self, ConversionError<T::FatalError>> {
         match T::from_pest(pest) {
-            Err(ConversionError::NoMatch) | Err(ConversionError::NoMatchWithInfo { .. }) => Ok(None),
+            Err(ref e) if e.is_no_match() => Ok(None),
             result => result.map(Some),
         }
     }
@@ -156,9 +156,7 @@ impl<'pest, Rule: RuleType, T: FromPest<'pest, Rule = Rule>> FromPest<'pest> for
         loop {
             match T::from_pest(pest) {
                 Ok(t) => acc.push(t),
-                Err(ConversionError::NoMatch) | Err(ConversionError::NoMatchWithInfo { .. }) => {
-                    break
-                }
+                Err(ref e) if e.is_no_match() => break,
                 Err(error) => return Err(error),
             }
         }