Address code review feedback

- Extract helper function `is_one_sided_glob_pattern` for better readability
- Change `glob_count >= 1` to `glob_count > 0` with clarifying comment
- All tests still passing

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

Author: Copilot

gix-refspec/src/parse.rs

@@ -161,7 +161,8 @@ pub(crate) mod function {
                         return Err(Error::PatternUnsupported { pattern: spec.into() });
                     }
                 }
-                let has_globs = glob_count >= 1;
+                // Check if there are any globs (one or more asterisks)
+                let has_globs = glob_count > 0;
                 if has_globs {
                     // For one-sided refspecs, skip validation of glob patterns
                     if !is_one_sided {

gix-refspec/tests/refspec/parse/mod.rs

@@ -46,7 +46,7 @@ fn baseline() {
                             true,
                         ) => {} // we prefer failing fast, git let's it pass
                         // We now allow complex glob patterns in one-sided refspecs
-                        (None, false) if matches!(op, Operation::Fetch) && spec.to_str().map(|s| s.contains('*') && !s.contains(':')).unwrap_or(false) => {
+                        (None, false) if is_one_sided_glob_pattern(spec, op) => {
                             // This is an intentional behavior change: we allow complex globs in one-sided refspecs
                         }
                         _ => {
@@ -70,6 +70,11 @@ fn baseline() {
             panics
         );
     }
+    
+    fn is_one_sided_glob_pattern(spec: &[u8], op: Operation) -> bool {
+        use bstr::ByteSlice;
+        matches!(op, Operation::Fetch) && spec.to_str().map(|s| s.contains('*') && !s.contains(':')).unwrap_or(false)
+    }
 }
 
 #[test]