From 089215c1c26b4b09f697e3711416cbed0c53b0d3 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
Date: Mon, 17 Aug 2026 02:19:11 +0900
Subject: refactor(preg): drop unused flags and offset of preg_match_all2()

Every caller reached preg_match_all2() through Preg::match_all5(), which
always passed flags = PREG_UNMATCHED_AS_NULL and offset = 0. Inline those
constants and make match_all5() private, since the two public wrappers are
its only callers.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 crates/shirabe-pcre/src/preg.rs | 21 +++++----------------
 1 file changed, 5 insertions(+), 16 deletions(-)

(limited to 'crates/shirabe-pcre/src')

diff --git a/crates/shirabe-pcre/src/preg.rs b/crates/shirabe-pcre/src/preg.rs
index e197c874..95697e83 100644
--- a/crates/shirabe-pcre/src/preg.rs
+++ b/crates/shirabe-pcre/src/preg.rs
@@ -57,7 +57,7 @@ impl Preg {
     }
 
     pub fn match_all(pattern: impl PregPattern, subject: &str) -> usize {
-        Self::match_all5(pattern, subject, None, 0, 0)
+        Self::match_all5(pattern, subject, None)
     }
 
     pub fn match_all3(
@@ -65,27 +65,16 @@ impl Preg {
         subject: &str,
         matches: Option<&mut IndexMap<CaptureKey, Vec<String>>>,
     ) -> usize {
-        Self::match_all5(pattern, subject, matches, 0, 0)
+        Self::match_all5(pattern, subject, matches)
     }
 
-    pub fn match_all5(
+    fn match_all5(
         pattern: impl PregPattern,
         subject: &str,
         matches: Option<&mut IndexMap<CaptureKey, Vec<String>>>,
-        flags: i64,
-        offset: usize,
     ) -> usize {
-        Self::check_offset_capture(flags, "matchAllWithOffsets");
-        Self::check_set_order(flags);
-
         let mut internal: IndexMap<CaptureKey, Vec<Option<String>>> = IndexMap::new();
-        let result = preg_match_all2(
-            pattern,
-            subject,
-            &mut internal,
-            flags | PREG_UNMATCHED_AS_NULL,
-            offset,
-        );
+        let result = preg_match_all2(pattern, subject, &mut internal);
 
         if let Some(out) = matches {
             *out = null_to_empty_match_all(internal);
@@ -266,7 +255,7 @@ impl Preg {
         subject: &str,
         matches: Option<&mut IndexMap<CaptureKey, Vec<String>>>,
     ) -> bool {
-        Self::match_all5(pattern, subject, matches, 0, 0) > 0
+        Self::match_all5(pattern, subject, matches) > 0
     }
 
     pub fn is_match_all_with_offsets3(
-- 
cgit v1.3.1-4-g156e