From 6d721b21a1b910448885c4540eb37a5398e48e11 Mon Sep 17 00:00:00 2001 From: Niels Dossche <7771979+nielsdos@users.noreply.github.com> Date: Sun, 25 Feb 2024 16:23:00 +0100 Subject: [PATCH] Use string literal equality helper function in xmlreader_get_method --- ext/xmlreader/php_xmlreader.c | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/ext/xmlreader/php_xmlreader.c b/ext/xmlreader/php_xmlreader.c index 3221dbb34b6..f56707c0e7f 100644 --- a/ext/xmlreader/php_xmlreader.c +++ b/ext/xmlreader/php_xmlreader.c @@ -181,19 +181,12 @@ zval *xmlreader_write_property(zend_object *object, zend_string *name, zval *val /* {{{ */ static zend_function *xmlreader_get_method(zend_object **obj, zend_string *name, const zval *key) { - if (ZSTR_LEN(name) == sizeof("open") - 1 - && (ZSTR_VAL(name)[0] == 'o' || ZSTR_VAL(name)[0] == 'O') - && (ZSTR_VAL(name)[1] == 'p' || ZSTR_VAL(name)[1] == 'P') - && (ZSTR_VAL(name)[2] == 'e' || ZSTR_VAL(name)[2] == 'E') - && (ZSTR_VAL(name)[3] == 'n' || ZSTR_VAL(name)[3] == 'N')) { + if (zend_string_equals_literal_ci(name, "open")) { return (zend_function*)&xmlreader_open_fn; - } else if (ZSTR_LEN(name) == sizeof("xml") - 1 - && (ZSTR_VAL(name)[0] == 'x' || ZSTR_VAL(name)[0] == 'X') - && (ZSTR_VAL(name)[1] == 'm' || ZSTR_VAL(name)[1] == 'M') - && (ZSTR_VAL(name)[2] == 'l' || ZSTR_VAL(name)[2] == 'L')) { + } else if (zend_string_equals_literal_ci(name, "xml")) { return (zend_function*)&xmlreader_xml_fn; } - return zend_std_get_method(obj, name, key);; + return zend_std_get_method(obj, name, key); } /* }}} */