- Interned string related callbacks moved turned from compiler_globals into real...

- Updated API version number

diff --git a/Zend/zend_API.c b/Zend/zend_API.c
index d2b5f78fc3ca46e1a25fc09653301e75536efd68..8077fbba5425f88d1c314ee7fc0f949115744d93 100644 (file)
--- a/Zend/zend_API.c
+++ b/Zend/zend_API.c
@@ -1917,7 +1917,7 @@ ZEND_API int zend_register_functions(zend_class_entry *scope, const zend_functio
                        }
                }
                fname_len = strlen(ptr->fname);
-               lowercase_name = CG(new_interned_string)(zend_str_tolower_dup(ptr->fname, fname_len), fname_len + 1, 1 TSRMLS_CC);
+               lowercase_name = zend_new_interned_string(zend_str_tolower_dup(ptr->fname, fname_len), fname_len + 1, 1 TSRMLS_CC);
                if (zend_hash_add(target_function_table, lowercase_name, fname_len+1, &function, sizeof(zend_function), (void**)&reg_function) == FAILURE) {
                        unload=1;
                        str_efree(lowercase_name);
@@ -2205,7 +2205,7 @@ static zend_class_entry *do_register_internal_class(zend_class_entry *orig_class
        }
 
        zend_str_tolower_copy(lowercase_name, orig_class_entry->name, class_entry->name_length);
-       lowercase_name = CG(new_interned_string)(lowercase_name, class_entry->name_length + 1, 1 TSRMLS_CC);
+       lowercase_name = zend_new_interned_string(lowercase_name, class_entry->name_length + 1, 1 TSRMLS_CC);
        zend_hash_update(CG(class_table), lowercase_name, class_entry->name_length+1, &class_entry, sizeof(zend_class_entry *), NULL);
        str_efree(lowercase_name);
        return class_entry;
@@ -3165,7 +3165,7 @@ ZEND_API int zend_declare_property_ex(zend_class_entry *ce, const char *name, in
                        break;
        }
 
-       interned_name = CG(new_interned_string)(property_info.name, property_info.name_length+1, 0 TSRMLS_CC);
+       interned_name = zend_new_interned_string(property_info.name, property_info.name_length+1, 0 TSRMLS_CC);
        if (interned_name != property_info.name) {
                if (ce->type == ZEND_USER_CLASS) {
                        efree(property_info.name);

diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c
index 5d49f1c5e5cdfd04703bf2e903f700f4d27a6ce0..37ee831bf2d00d7fee50e007966cfcf5572c7449 100644 (file)
--- a/Zend/zend_compile.c
+++ b/Zend/zend_compile.c
@@ -343,7 +343,7 @@ static int lookup_cv(zend_op_array *op_array, char* name, int name_len TSRMLS_DC
                op_array->size_var += 16; /* FIXME */
                op_array->vars = erealloc(op_array->vars, op_array->size_var*sizeof(zend_compiled_variable));
        }
-       op_array->vars[i].name = CG(new_interned_string)(name, name_len + 1, 1 TSRMLS_CC);
+       op_array->vars[i].name = zend_new_interned_string(name, name_len + 1, 1 TSRMLS_CC);
        op_array->vars[i].name_len = name_len;
        op_array->vars[i].hash_value = hash_value;
        return i;
@@ -374,7 +374,7 @@ int zend_add_literal(zend_op_array *op_array, const zval *zv) /* {{{ */
                TSRMLS_FETCH();
 
                Z_STRVAL_P(z) =
-                       CG(new_interned_string)(Z_STRVAL_P(zv), Z_STRLEN_P(zv) + 1, 1 TSRMLS_CC);
+                       zend_new_interned_string(Z_STRVAL_P(zv), Z_STRLEN_P(zv) + 1, 1 TSRMLS_CC);
        }
        CONSTANT_EX(op_array, i) = *zv;
        Z_SET_REFCOUNT(CONSTANT_EX(op_array, i), 2);
@@ -1540,7 +1540,7 @@ void zend_do_begin_function_declaration(znode *function_token, znode *function_n
        op_array.line_start = zend_get_compiled_lineno(TSRMLS_C);
 
        if (is_method) {
-               lcname = CG(new_interned_string)(zend_str_tolower_dup(name, name_len), name_len + 1, 1 TSRMLS_CC);
+               lcname = zend_new_interned_string(zend_str_tolower_dup(name, name_len), name_len + 1, 1 TSRMLS_CC);
 
                if (zend_hash_add(&CG(active_class_entry)->function_table, lcname, name_len+1, &op_array, sizeof(zend_op_array), (void **) &CG(active_op_array)) == FAILURE) {
                        zend_error(E_COMPILE_ERROR, "Cannot redeclare %s::%s()", CG(active_class_entry)->name, name);
@@ -1836,7 +1836,7 @@ void zend_do_receive_arg(zend_uchar op, znode *varname, const znode *offset, con
        }
        CG(active_op_array)->arg_info = erealloc(CG(active_op_array)->arg_info, sizeof(zend_arg_info)*(CG(active_op_array)->num_args));
        cur_arg_info = &CG(active_op_array)->arg_info[CG(active_op_array)->num_args-1];
-       cur_arg_info->name = CG(new_interned_string)(estrndup(varname->u.constant.value.str.val, varname->u.constant.value.str.len), varname->u.constant.value.str.len + 1, 1 TSRMLS_CC);
+       cur_arg_info->name = zend_new_interned_string(estrndup(varname->u.constant.value.str.val, varname->u.constant.value.str.len), varname->u.constant.value.str.len + 1, 1 TSRMLS_CC);
        cur_arg_info->name_len = varname->u.constant.value.str.len;
        cur_arg_info->type_hint = 0;
        cur_arg_info->allow_null = 1;
@@ -1853,7 +1853,7 @@ void zend_do_receive_arg(zend_uchar op, znode *varname, const znode *offset, con
                        if (ZEND_FETCH_CLASS_DEFAULT == zend_get_class_fetch_type(Z_STRVAL(class_type->u.constant), Z_STRLEN(class_type->u.constant))) {
                                zend_resolve_class_name(class_type, &opline->extended_value, 1 TSRMLS_CC);
                        }
-                       class_type->u.constant.value.str.val = CG(new_interned_string)(class_type->u.constant.value.str.val, class_type->u.constant.value.str.len + 1, 1 TSRMLS_CC);
+                       class_type->u.constant.value.str.val = zend_new_interned_string(class_type->u.constant.value.str.val, class_type->u.constant.value.str.len + 1, 1 TSRMLS_CC);
                        cur_arg_info->class_name = class_type->u.constant.value.str.val;
                        cur_arg_info->class_name_len = class_type->u.constant.value.str.len;
                        if (op == ZEND_RECV_INIT) {
@@ -4766,7 +4766,7 @@ void zend_do_declare_property(const znode *var_name, const znode *value, zend_ui
                CG(doc_comment_len) = 0;
        }
 
-       zend_declare_property_ex(CG(active_class_entry), CG(new_interned_string)(var_name->u.constant.value.str.val, var_name->u.constant.value.str.len + 1, 0 TSRMLS_CC), var_name->u.constant.value.str.len, property, access_type, comment, comment_len TSRMLS_CC);
+       zend_declare_property_ex(CG(active_class_entry), zend_new_interned_string(var_name->u.constant.value.str.val, var_name->u.constant.value.str.len + 1, 0 TSRMLS_CC), var_name->u.constant.value.str.len, property, access_type, comment, comment_len TSRMLS_CC);
        efree(var_name->u.constant.value.str.val);
 }
 /* }}} */
@@ -4787,7 +4787,7 @@ void zend_do_declare_class_constant(znode *var_name, const znode *value TSRMLS_D
        ALLOC_ZVAL(property);
        *property = value->u.constant;
 
-       if (zend_hash_add(&CG(active_class_entry)->constants_table, CG(new_interned_string)(var_name->u.constant.value.str.val, var_name->u.constant.value.str.len+1, 0 TSRMLS_CC), var_name->u.constant.value.str.len+1, &property, sizeof(zval *), NULL)==FAILURE) {
+       if (zend_hash_add(&CG(active_class_entry)->constants_table, zend_new_interned_string(var_name->u.constant.value.str.val, var_name->u.constant.value.str.len+1, 0 TSRMLS_CC), var_name->u.constant.value.str.len+1, &property, sizeof(zval *), NULL)==FAILURE) {
                FREE_ZVAL(property);
                zend_error(E_COMPILE_ERROR, "Cannot redefine class constant %s::%s", CG(active_class_entry)->name, var_name->u.constant.value.str.val);
        }

diff --git a/Zend/zend_constants.c b/Zend/zend_constants.c
index 73ebec9a33f237fa424cf37b71735cbe21b9e00a..ed00b993491e0970ca2b4fea3cdc8e048104dd24 100644 (file)
--- a/Zend/zend_constants.c
+++ b/Zend/zend_constants.c
@@ -459,14 +459,14 @@ ZEND_API int zend_register_constant(zend_constant *c TSRMLS_DC)
                /* keep in mind that c->name_len already contains the '\0' */
                lowercase_name = estrndup(c->name, c->name_len-1);
                zend_str_tolower(lowercase_name, c->name_len-1);
-               lowercase_name = CG(new_interned_string)(lowercase_name, c->name_len, 1 TSRMLS_CC);
+               lowercase_name = zend_new_interned_string(lowercase_name, c->name_len, 1 TSRMLS_CC);
                name = lowercase_name;
        } else {
                char *slash = strrchr(c->name, '\\');
                if(slash) {
                        lowercase_name = estrndup(c->name, c->name_len-1);
                        zend_str_tolower(lowercase_name, slash-c->name);
-                       lowercase_name = CG(new_interned_string)(lowercase_name, c->name_len, 1 TSRMLS_CC);
+                       lowercase_name = zend_new_interned_string(lowercase_name, c->name_len, 1 TSRMLS_CC);
                        name = lowercase_name;
                } else {
                        name = c->name;

diff --git a/Zend/zend_extensions.h b/Zend/zend_extensions.h
index b08a8bfc84e10a3df5978d018d0578809388036e..f827febf8453b01bcd1289f51d9307f20dd10f3d 100644 (file)
--- a/Zend/zend_extensions.h
+++ b/Zend/zend_extensions.h
@@ -28,7 +28,7 @@
 /* The first number is the engine version and the rest is the date.
  * This way engine 2/3 API no. is always greater than engine 1 API no..
  */
-#define ZEND_EXTENSION_API_NO  220100409
+#define ZEND_EXTENSION_API_NO  220100525
 
 typedef struct _zend_extension_version_info {
        int zend_extension_api_no;

diff --git a/Zend/zend_globals.h b/Zend/zend_globals.h
index 01f24712c1ec37e7dc29b405aace80bf0383126a..f8bc38c20d03e323bb4d8f7a7da3680b82545dea 100644 (file)
--- a/Zend/zend_globals.h
+++ b/Zend/zend_globals.h
@@ -147,10 +147,6 @@ struct _zend_compiler_globals {
 
        HashTable interned_strings;
 
-       char *(*new_interned_string)(char *str, int len, int free_src TSRMLS_DC);
-       void (*interned_strings_snapshot)(TSRMLS_D);
-       void (*interned_strings_restore)(TSRMLS_D);
-
 #ifdef ZEND_MULTIBYTE
        zend_encoding **script_encoding_list;
        size_t script_encoding_list_size;

diff --git a/Zend/zend_modules.h b/Zend/zend_modules.h
index 915634a56f08dd062d004894d110723c13cd525d..32f29c09b7e05b111e551eb5caf0417e1f1d81a8 100644 (file)
--- a/Zend/zend_modules.h
+++ b/Zend/zend_modules.h
@@ -33,7 +33,7 @@
 #define ZEND_MODULE_INFO_FUNC_ARGS zend_module_entry *zend_module TSRMLS_DC
 #define ZEND_MODULE_INFO_FUNC_ARGS_PASSTHRU zend_module TSRMLS_CC
 
-#define ZEND_MODULE_API_NO 20100409
+#define ZEND_MODULE_API_NO 20100525
 #ifdef ZTS
 #define USING_ZTS 1
 #else

diff --git a/Zend/zend_string.c b/Zend/zend_string.c
index 098ad62713a54a4ed21207160da002fabb08728f..99b24f64e94773cd89827be4b372537de6112da6 100644 (file)
--- a/Zend/zend_string.c
+++ b/Zend/zend_string.c
@@ -29,9 +29,13 @@
 # include <sys/mman.h>
 #endif
 
-static char *zend_new_interned_string(char *str, int len, int free_src TSRMLS_DC);
-static void zend_interned_strings_snapshot(TSRMLS_D);
-static void zend_interned_strings_restore(TSRMLS_D);
+ZEND_API char *(*zend_new_interned_string)(char *str, int len, int free_src TSRMLS_DC);
+ZEND_API void (*zend_interned_strings_snapshot)(TSRMLS_D);
+ZEND_API void (*zend_interned_strings_restore)(TSRMLS_D);
+
+static char *zend_new_interned_string_int(char *str, int len, int free_src TSRMLS_DC);
+static void zend_interned_strings_snapshot_int(TSRMLS_D);
+static void zend_interned_strings_restore_int(TSRMLS_D);
 
 void zend_interned_strings_init(TSRMLS_D)
 {
@@ -59,9 +63,9 @@ void zend_interned_strings_init(TSRMLS_D)
 
 #endif
 
-       CG(new_interned_string) = zend_new_interned_string;
-       CG(interned_strings_snapshot) = zend_interned_strings_snapshot;
-       CG(interned_strings_restore) = zend_interned_strings_restore;
+       zend_new_interned_string = zend_new_interned_string_int;
+       zend_interned_strings_snapshot = zend_interned_strings_snapshot_int;
+       zend_interned_strings_restore = zend_interned_strings_restore_int;
 }
 
 void zend_interned_strings_dtor(TSRMLS_D)
@@ -75,7 +79,7 @@ void zend_interned_strings_dtor(TSRMLS_D)
 #endif
 }
 
-static char *zend_new_interned_string(char *arKey, int nKeyLength, int free_src TSRMLS_DC)
+static char *zend_new_interned_string_int(char *arKey, int nKeyLength, int free_src TSRMLS_DC)
 {
 #ifndef ZTS
        ulong h;
@@ -173,12 +177,12 @@ static char *zend_new_interned_string(char *arKey, int nKeyLength, int free_src
 #endif
 }
 
-static void zend_interned_strings_snapshot(TSRMLS_D)
+static void zend_interned_strings_snapshot_int(TSRMLS_D)
 {
        CG(interned_strings_snapshot_top) = CG(interned_strings_top);
 }
 
-static void zend_interned_strings_restore(TSRMLS_D)
+static void zend_interned_strings_restore_int(TSRMLS_D)
 {
 #ifndef ZTS
        Bucket *p;

diff --git a/Zend/zend_string.h b/Zend/zend_string.h
index 158efceeb83853420698dd15883fa0267511de34..f657b4f14de8f9247a0bf56f03515c6b8556a93a 100644 (file)
--- a/Zend/zend_string.h
+++ b/Zend/zend_string.h
@@ -23,6 +23,10 @@
 
 #include "zend.h"
 
+ZEND_API extern char *(*zend_new_interned_string)(char *str, int len, int free_src TSRMLS_DC);
+ZEND_API extern void (*zend_interned_strings_snapshot)(TSRMLS_D);
+ZEND_API extern void (*zend_interned_strings_restore)(TSRMLS_D);
+
 void zend_interned_strings_init(TSRMLS_D);
 void zend_interned_strings_dtor(TSRMLS_D);
 

diff --git a/main/main.c b/main/main.c
index 3de188e0beb3e8b852489c2364d49cdb91d61ec8..f975c7f815eb18dd778a5bedb0b0666196cb6f59 100644 (file)
--- a/main/main.c
+++ b/main/main.c
@@ -1544,7 +1544,7 @@ void php_request_shutdown_for_exec(void *dummy)
        /* used to close fd's in the 3..255 range here, but it's problematic
         */
        shutdown_memory_manager(1, 1 TSRMLS_CC);
-       CG(interned_strings_restore)(TSRMLS_C);
+       zend_interned_strings_restore(TSRMLS_C);
 }
 /* }}} */
 
@@ -1587,7 +1587,7 @@ void php_request_shutdown_for_hook(void *dummy)
                shutdown_memory_manager(CG(unclean_shutdown), 0 TSRMLS_CC);
        } zend_end_try();
 
-       CG(interned_strings_restore)(TSRMLS_C);
+       zend_interned_strings_restore(TSRMLS_C);
 
        zend_try {
                zend_unset_timeout(TSRMLS_C);
@@ -1687,7 +1687,7 @@ void php_request_shutdown(void *dummy)
        zend_try {
                shutdown_memory_manager(CG(unclean_shutdown) || !report_memleaks, 0 TSRMLS_CC);
        } zend_end_try();
-       CG(interned_strings_restore)(TSRMLS_C);
+       zend_interned_strings_restore(TSRMLS_C);
 
        /* 12. Reset max_execution_time */
        zend_try {
@@ -2154,7 +2154,7 @@ int php_module_startup(sapi_module_struct *sf, zend_module_entry *additional_mod
        module_startup = 0;
 
        shutdown_memory_manager(1, 0 TSRMLS_CC);
-       CG(interned_strings_snapshot)(TSRMLS_C);
+       zend_interned_strings_snapshot(TSRMLS_C);
 
        /* we're done */
        return SUCCESS;