- Added zend_optimizerplus.load_comments configuration durective, so "Doc comments...

restored only by applicaions that really need them.
- Added note, that zend_optimizerplus.save_comments is dangerous and may break applications and frameworks.

diff --git a/README b/README
index 05c4c6068139a14496c51be625f46d173337c0ac..5b25a7aa399116452c620a3501fc76f790bdd87e 100644 (file)
--- a/README
+++ b/README
@@ -44,11 +44,16 @@ zend_optimizerplus.memory_consumption=128
 zend_optimizerplus.interned_strings_buffer=8
 zend_optimizerplus.max_accelerated_files=4000
 zend_optimizerplus.revalidate_freq=60
-zend_optimizerplus.save_comments=0
 zend_optimizerplus.fast_shutdown=1
-zend_optimizerplus.enable_file_override=1
 zend_optimizerplus.enable_cli=1
 
+You also may add the following, but it may break some applications and
+frameworks. Please, read description of these directives and add them on your
+own risk.
+
+zend_optimizerplus.save_comments=0
+zend_optimizerplus.enable_file_override=1
+
 In some cases you may like to prefer enabling/disabling some features
 to avoid incompatibilities at the cost of some performance degradation.
 
@@ -102,7 +107,13 @@ zend_optimizerplus.revalidate_path (default "0")
 
 zend_optimizerplus.save_comments (default "1")
        If disabled, all PHPDoc comments are dropped from the code to reduce the
-       size of the optimized code.
+       size of the optimized code. Disabling "Doc Comments" may break some
+       existing applications and frameworks (e.g. Doctrine, ZF2, PHPUnit)
+
+zend_optimizerplus.load_comments (default "1")
+       If disabled, PHPDoc comments are not loaded from SHM, so "Doc Comments"
+       may be always stored (save_comments=1), but not loaded by applications
+       that don't need them anyway.
 
 zend_optimizerplus.fast_shutdown (default "0")
        If enabled, a fast shutdown sequence is used for the accelerated code

diff --git a/ZendAccelerator.h b/ZendAccelerator.h
index 2aa2248cd3240b9b21b1a16c9fd237938aad42e5..d7ac0ed388473b9a58e9bee445cdaadd4c3f508e 100644 (file)
--- a/ZendAccelerator.h
+++ b/ZendAccelerator.h
@@ -204,6 +204,7 @@ typedef struct _zend_accel_directives {
        zend_bool      validate_timestamps;
        zend_bool      revalidate_path;
        zend_bool      save_comments;
+       zend_bool      load_comments;
        zend_bool      fast_shutdown;
        zend_bool      protect_memory;
        zend_bool      file_override_enabled;

diff --git a/zend_accelerator_module.c b/zend_accelerator_module.c
index be0655adf342ade7a35cf7607c628165f906d504..e64ad19e3c8a589d19e160d4bd4ca39e53d315b8 100644 (file)
--- a/zend_accelerator_module.c
+++ b/zend_accelerator_module.c
@@ -221,6 +221,7 @@ ZEND_INI_BEGIN()
 
        STD_PHP_INI_ENTRY("zend_optimizerplus.protect_memory"        , "0"  , PHP_INI_SYSTEM, OnUpdateBool,                  accel_directives.protect_memory,            zend_accel_globals, accel_globals)
        STD_PHP_INI_ENTRY("zend_optimizerplus.save_comments"         , "1"  , PHP_INI_SYSTEM, OnUpdateBool,                  accel_directives.save_comments,             zend_accel_globals, accel_globals)
+       STD_PHP_INI_ENTRY("zend_optimizerplus.load_comments"         , "1"  , PHP_INI_ALL,    OnUpdateBool,                  accel_directives.load_comments,             zend_accel_globals, accel_globals)
        STD_PHP_INI_ENTRY("zend_optimizerplus.fast_shutdown"         , "0"  , PHP_INI_SYSTEM, OnUpdateBool,                  accel_directives.fast_shutdown,             zend_accel_globals, accel_globals)
 
        STD_PHP_INI_ENTRY("zend_optimizerplus.optimization_level"    , DEFAULT_OPTIMIZATION_LEVEL , PHP_INI_SYSTEM, OnUpdateLong, accel_directives.optimization_level,   zend_accel_globals, accel_globals)

diff --git a/zend_accelerator_util_funcs.c b/zend_accelerator_util_funcs.c
index 275642bd218e9c055937eabee205461c223c0813..c41080fbd346a0254223d8d8e088ac8458013f6e 100644 (file)
--- a/zend_accelerator_util_funcs.c
+++ b/zend_accelerator_util_funcs.c
@@ -520,7 +520,11 @@ static void zend_hash_clone_prop_info(HashTable *ht, HashTable *source, zend_cla
                /* Copy constructor */
                prop_info->name = interned_estrndup(prop_info->name, prop_info->name_length);
                if(prop_info->doc_comment) {
-                       prop_info->doc_comment = estrndup(prop_info->doc_comment, prop_info->doc_comment_len);
+                       if (ZCG(accel_directives).load_comments) {
+                               prop_info->doc_comment = estrndup(prop_info->doc_comment, prop_info->doc_comment_len);
+                       } else {
+                               prop_info->doc_comment = NULL;
+                       }
                }
                if(prop_info->ce == old_ce) {
                        prop_info->ce = ce;
@@ -637,7 +641,11 @@ static void zend_class_copy_ctor(zend_class_entry **pce)
                ce->interfaces = NULL;
        }
        if (ZEND_CE_DOC_COMMENT(ce)) {
-               ZEND_CE_DOC_COMMENT(ce) = estrndup(ZEND_CE_DOC_COMMENT(ce), ZEND_CE_DOC_COMMENT_LEN(ce));
+               if (ZCG(accel_directives).load_comments) {
+                       ZEND_CE_DOC_COMMENT(ce) = estrndup(ZEND_CE_DOC_COMMENT(ce), ZEND_CE_DOC_COMMENT_LEN(ce));
+               } else {
+                       ZEND_CE_DOC_COMMENT(ce) =  NULL;
+               }
        }
 
        if(ce->parent) {