Address review comments

Author: TysonAndre

Zend/zend_compile.c

@@ -1941,7 +1941,7 @@ ZEND_API size_t zend_dirname(char *path, size_t len)
 	/* Note that on Win32 CWD is per drive (heritage from CP/M).
 	 * This means dirname("c:foo") maps to "c:." or "c:" - which means CWD on C: drive.
 	 */
-	if ((2 <= len) && zend_isalpha_ascii((int)((unsigned char *)path)[0]) && (':' == path[1])) {
+	if ((2 <= len) && zend_isalpha_ascii(((unsigned char *)path)[0]) && (':' == path[1])) {
 		/* Skip over the drive spec (if any) so as not to change */
 		path += 2;
 		len_adjust += 2;

Zend/zend_operators.h

@@ -421,7 +421,7 @@ static zend_always_inline bool zend_isalpha_ascii(unsigned char c) {
 	/* Returns true for a-z and A-Z in a locale-independent way.
 	 * This is implemented in a way that can avoid branching. Note that ASCII 'a' == 'A' | 0x20. */
 	c = (c | 0x20) - 'a';
-	return c <= ('z' - 'a' + 1);
+	return c <= ('z' - 'a');
 }
 #define zend_isalnum_ascii(c) (zend_isalnum_map[(unsigned char)(c)])