Skip to content

Commit 4d16463

Browse files
committed
no magic nrs
1 parent 3c8b68e commit 4d16463

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

ext/json/json_encoder.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -523,7 +523,7 @@ zend_result php_json_escape_string(
523523
/* signed compare, so checks for unsigned ASCII >= 0x80 as well */
524524
const __m128i input_range = _mm_cmplt_epi8(input, _mm_set1_epi8(32));
525525

526-
int max_shift = 16;
526+
int max_shift = sizeof(__m128i);
527527

528528
int input_range_mask = _mm_movemask_epi8(input_range);
529529
if (input_range_mask != 0) {
@@ -554,7 +554,7 @@ zend_result php_json_escape_string(
554554
int mask = _mm_movemask_epi8(result_individual_bytes);
555555
#endif
556556
if (mask != 0) {
557-
if (max_shift < 16) {
557+
if (max_shift < sizeof(__m128i)) {
558558
int shift = zend_ulong_ntz(mask); /* first offending character */
559559
pos += MIN(max_shift, shift);
560560
len -= MIN(max_shift, shift);
@@ -580,9 +580,9 @@ zend_result php_json_escape_string(
580580
ZEND_ASSERT(handled == true);
581581
} while (mask != 0);
582582

583-
pos = 16 - (s - s_backup);
583+
pos = sizeof(__m128i) - (s - s_backup);
584584
} else {
585-
if (max_shift < 16) {
585+
if (max_shift < sizeof(__m128i)) {
586586
pos += max_shift;
587587
len -= max_shift;
588588
break;

0 commit comments

Comments
 (0)