Skip to content

Commit 8f78498

Browse files
committed
feedback
1 parent 630dff8 commit 8f78498

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

ext/pcre/php_pcre.c

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1245,13 +1245,12 @@ PHPAPI void php_pcre_match_impl(pcre_cache_entry *pce, zend_string *subject_str,
12451245

12461246
old_mdata_used = mdata_used;
12471247
if (!old_mdata_used && num_subpats <= PHP_PCRE_PREALLOC_MDATA_SIZE) {
1248-
mdata_used = 1;
1248+
mdata_used = true;
12491249
match_data = mdata;
12501250
} else {
12511251
match_data = pcre2_match_data_create_from_pattern(pce->re, PCRE_G(gctx_zmm));
12521252
if (!match_data) {
12531253
PCRE_G(error_code) = PHP_PCRE_INTERNAL_ERROR;
1254-
mdata_used = old_mdata_used;
12551254
RETURN_FALSE;
12561255
}
12571256
}
@@ -1653,13 +1652,12 @@ PHPAPI zend_string *php_pcre_replace_impl(pcre_cache_entry *pce, zend_string *su
16531652

16541653
old_mdata_used = mdata_used;
16551654
if (!old_mdata_used && num_subpats <= PHP_PCRE_PREALLOC_MDATA_SIZE) {
1656-
mdata_used = 1;
1655+
mdata_used = true;
16571656
match_data = mdata;
16581657
} else {
16591658
match_data = pcre2_match_data_create_from_pattern(pce->re, PCRE_G(gctx_zmm));
16601659
if (!match_data) {
16611660
PCRE_G(error_code) = PHP_PCRE_INTERNAL_ERROR;
1662-
mdata_used = old_mdata_used;
16631661
return NULL;
16641662
}
16651663
}
@@ -2614,13 +2612,12 @@ PHPAPI void php_pcre_split_impl(pcre_cache_entry *pce, zend_string *subject_str,
26142612

26152613
old_mdata_used = mdata_used;
26162614
if (!old_mdata_used && num_subpats <= PHP_PCRE_PREALLOC_MDATA_SIZE) {
2617-
mdata_used = 1;
2615+
mdata_used = true;
26182616
match_data = mdata;
26192617
} else {
26202618
match_data = pcre2_match_data_create_from_pattern(pce->re, PCRE_G(gctx_zmm));
26212619
if (!match_data) {
26222620
PCRE_G(error_code) = PHP_PCRE_INTERNAL_ERROR;
2623-
mdata_used = old_mdata_used;
26242621
zval_ptr_dtor(return_value);
26252622
RETURN_FALSE;
26262623
}
@@ -2959,13 +2956,12 @@ PHPAPI void php_pcre_grep_impl(pcre_cache_entry *pce, zval *input, zval *return
29592956

29602957
old_mdata_used = mdata_used;
29612958
if (!old_mdata_used && num_subpats <= PHP_PCRE_PREALLOC_MDATA_SIZE) {
2962-
mdata_used = 1;
2959+
mdata_used = true;
29632960
match_data = mdata;
29642961
} else {
29652962
match_data = pcre2_match_data_create_from_pattern(pce->re, PCRE_G(gctx_zmm));
29662963
if (!match_data) {
29672964
PCRE_G(error_code) = PHP_PCRE_INTERNAL_ERROR;
2968-
mdata_used = old_mdata_used;
29692965
return;
29702966
}
29712967
}

0 commit comments

Comments
 (0)