summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitry Stogov <dmitry@php.net>2010-07-09 07:31:18 +0000
committerDmitry Stogov <dmitry@php.net>2010-07-09 07:31:18 +0000
commitbc1f1d25fa30805a8f4208829cad241880a486ab (patch)
tree4528a8b65d4a1d6d59a176c555196bee85a6fb57
parentb458b1a4c1380cdc2891f2d9827fa770aab878d7 (diff)
downloadphp-git-bc1f1d25fa30805a8f4208829cad241880a486ab.tar.gz
zend_ptr_stack allocation is delayed before the actual usage
-rw-r--r--NEWS2
-rw-r--r--Zend/zend_ptr_stack.c5
-rw-r--r--Zend/zend_ptr_stack.h5
3 files changed, 6 insertions, 6 deletions
diff --git a/NEWS b/NEWS
index 712366b66d..000f8972e7 100644
--- a/NEWS
+++ b/NEWS
@@ -18,7 +18,7 @@ PHP NEWS
A constant class name may be used as a direct operand of ZEND_FETCH_*
instruction without previous ZEND_FETCH_CLASS.
. eliminated unnecessary iterations during request startup/shutdown
- . zend_stack initialization is delayed before the actual usage
+ . zend_stack and zend_ptr_stack allocation is delayed before the actual usage
. $GLOBALS became a JIT autoglobal, so it's initialized only if used
(this may affect opcode caches)
- Added concept of interned strings. All strings constants known at compile
diff --git a/Zend/zend_ptr_stack.c b/Zend/zend_ptr_stack.c
index 0d4556d9ed..af345987f7 100644
--- a/Zend/zend_ptr_stack.c
+++ b/Zend/zend_ptr_stack.c
@@ -27,9 +27,8 @@
ZEND_API void zend_ptr_stack_init_ex(zend_ptr_stack *stack, zend_bool persistent)
{
- stack->top_element = stack->elements = (void **) pemalloc(sizeof(void *)*PTR_STACK_BLOCK_SIZE, persistent);
- stack->max = PTR_STACK_BLOCK_SIZE;
- stack->top = 0;
+ stack->top_element = stack->elements = NULL;
+ stack->top = stack->max = 0;
stack->persistent = persistent;
}
diff --git a/Zend/zend_ptr_stack.h b/Zend/zend_ptr_stack.h
index affe7d037b..8f2828d54a 100644
--- a/Zend/zend_ptr_stack.h
+++ b/Zend/zend_ptr_stack.h
@@ -46,8 +46,9 @@ END_EXTERN_C()
#define ZEND_PTR_STACK_RESIZE_IF_NEEDED(stack, count) \
if (stack->top+count > stack->max) { \
/* we need to allocate more memory */ \
- stack->max *= 2; \
- stack->max += count; \
+ do { \
+ stack->max += PTR_STACK_BLOCK_SIZE; \
+ } while (stack->top+count > stack->max); \
stack->elements = (void **) perealloc(stack->elements, (sizeof(void *) * (stack->max)), stack->persistent); \
stack->top_element = stack->elements+stack->top; \
}