com php-src: unnecessary 1 byte allocted: ext/opcache/Optimizer /compact_literals.c
| From: | Xinchen Hui | Date: | Fri, 21 Aug 2015 10:28:02 +0000 |
| Subject: | com php-src: unnecessary 1 byte allocted: ext/opcache/Optimizer /compact_literals.c | ||
| Groups: | php.cvs | ||
| Request: | Send a blank email to php-cvs+get-88716@lists.php.net to get a copy of this message | ||
Commit: 5a37056efae0a3ac7450b5ae36bce02db154c802
Author: Xinchen Hui <laruence@php.net> Fri, 21 Aug 2015 18:28:02 +0800
Parents: 13044447ebb5a86fa776eaf7625fe428064322b2
Branches: master
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=5a37056efae0a3ac7450b5ae36bce02db154c802
Log:
unnecessary 1 byte allocted
Changed paths:
M ext/opcache/Optimizer/compact_literals.c
Diff:
diff --git a/ext/opcache/Optimizer/compact_literals.c b/ext/opcache/Optimizer/compact_literals.c
index 50caf42..1b7e1d9 100644
--- a/ext/opcache/Optimizer/compact_literals.c
+++ b/ext/opcache/Optimizer/compact_literals.c
@@ -421,13 +421,13 @@ void zend_optimizer_compact_literals(zend_op_array *op_array,
zend_optimizer_ctx
case IS_CONSTANT:
if (info[i].flags & LITERAL_MAY_MERGE) {
if (info[i].flags & LITERAL_EX_OBJ) {
- int key_len = MAX_LENGTH_OF_LONG + sizeof("->") +
Z_STRLEN(op_array->literals[i]);
+ int key_len = MAX_LENGTH_OF_LONG + sizeof("->") - 1 +
Z_STRLEN(op_array->literals[i]);
key = zend_string_alloc(key_len, 0);
ZSTR_LEN(key) = snprintf(ZSTR_VAL(key), ZSTR_LEN(key)-1, "%d->%s",
info[i].u.num, Z_STRVAL(op_array->literals[i]));
} else if (info[i].flags & LITERAL_EX_CLASS) {
int key_len;
zval *class_name = &op_array->literals[(info[i].u.num < i) ? map[info[i].u.num] :
info[i].u.num];
- key_len = Z_STRLEN_P(class_name) + sizeof("::") +
Z_STRLEN(op_array->literals[i]);
+ key_len = Z_STRLEN_P(class_name) + sizeof("::") - 1 +
Z_STRLEN(op_array->literals[i]);
key = zend_string_alloc(key_len, 0);
memcpy(ZSTR_VAL(key), Z_STRVAL_P(class_name), Z_STRLEN_P(class_name));
memcpy(ZSTR_VAL(key) + Z_STRLEN_P(class_name), "::", sizeof("::") - 1);