Sablot extension cleanup - it compiles again now

diff --git a/ext/sablot/php_sablot.h b/ext/sablot/php_sablot.h
index 38f3dc0ef7e9e7ad957f379f92d04273a6c3fcba..f912944ac67f0473915c4958c780f03194f778c5 100644 (file)
--- a/ext/sablot/php_sablot.h
+++ b/ext/sablot/php_sablot.h
@@ -108,7 +108,7 @@ typedef struct {
 
 
 /* Sablotron Globals */
-typedef struct {
+typedef struct _php_sablot_globals {
     zval *errorHandler;
     php_sablot_error *errors;
     php_sablot_error errors_start;

diff --git a/ext/sablot/sablot.c b/ext/sablot/sablot.c
index 6547f9050e55b5d544b1e4128f4a96ea22bc2f02..268202f053497f203bc1624e77e10fa7061e80aa 100644 (file)
--- a/ext/sablot/sablot.c
+++ b/ext/sablot/sablot.c
@@ -166,12 +166,7 @@ static SchemeHandler sh = {
   _php_sablot_sh_close
 };
 
-#ifdef ZTS
-int sablot_globals_id;
-#else
-php_sablot_globals sablot_globals;
-#endif
-
+PHP_DECLARE_MODULE_GLOBALS(sablot)
 
 static unsigned char sixth_arg_force_ref[] = { 6, BYREF_NONE, BYREF_NONE, BYREF_NONE, BYREF_NONE, BYREF_NONE, BYREF_FORCE };
 static unsigned char third_arg_force_ref[] = { 4, BYREF_NONE, BYREF_NONE, BYREF_FORCE, BYREF_NONE };
@@ -214,12 +209,12 @@ zend_module_entry sablot_module_entry = {
 ZEND_GET_MODULE(sablot)
 #endif
 
-static void php_sablot_init_globals(php_sablot_init_globals *sablot_init_globals_p TSRMLS_DC)
+static void php_sablot_init_globals(php_sablot_globals *sablot_globals)
 {
-       SABLOTG(processor)             = NULL;
-       SABLOTG(errors)                = NULL;
-       SABLOTG(errorHandler)          = NULL;
-       SABLOTG(output_transform_file) = NULL;
+       sablot_globals->processor             = NULL;
+       sablot_globals->errors                = NULL;
+       sablot_globals->errorHandler          = NULL;
+       sablot_globals->output_transform_file = NULL;
 }
 
 
@@ -248,7 +243,9 @@ PHP_MSHUTDOWN_FUNCTION(sablot)
 
 PHP_RSHUTDOWN_FUNCTION(sablot)
 {
+       /*
        SABLOT_FREE_ERROR_HANDLE(SABLOTG_HANDLE);
+       */
        return SUCCESS;
 }
 
@@ -1349,7 +1346,9 @@ static MH_ERROR _php_sablot_error(void *userData, SablotHandle p, MH_ERROR code,
     TSRMLS_FETCH();
    
     if (userData == NULL) {
+               /* **FIXME** SABLOTG_HANDLE is not defined anywhere  -RL
         SABLOT_FREE_ERROR_HANDLE(SABLOTG_HANDLE);
+               */
         
         SABLOTG(errors_start).next = NULL;
         SABLOTG(errors)            = &SABLOTG(errors_start);

diff --git a/main/php.h b/main/php.h
index fb2f62f6662ecf0f8753dbdcdb61b5cfe0a34415..356be3dd5116994f6a99730ee908bf550935bf5d 100644 (file)
--- a/main/php.h
+++ b/main/php.h
@@ -228,6 +228,30 @@ char *strerror(int);
 #define PHP_RSHUTDOWN_FUNCTION(module) int PHP_RSHUTDOWN(module)(SHUTDOWN_FUNC_ARGS)
 #define PHP_MINFO_FUNCTION(module)     void PHP_MINFO(module)(ZEND_MODULE_INFO_FUNC_ARGS)
 
+#define PHP_BEGIN_MODULE_GLOBALS(module_name)      \
+    typedef struct _php_##module_name##_globals {
+#define PHP_END_MODULE_GLOBALS(module_name)        \
+                   } php_##module_name##_globals;
+
+#ifdef ZTS
+
+#define PHP_DECLARE_MODULE_GLOBALS(module_name)                            \
+    ts_rsrc_id module_name##_globals_id;
+#define PHP_EXTERN_MODULE_GLOBALS(module_name)                             \
+    extern ts_rsrc_id module_name##_globals_id;
+#define PHP_INIT_MODULE_GLOBALS(module_name, globals_ctor, globals_dtor)   \
+    ts_allocate_id(&module_name##_globals_id, sizeof(php_##module_name##_globals), (ts_allocate_ctor) globals_ctor, (ts_allocate_dtor) globals_dtor);
+
+#else
+
+#define PHP_DECLARE_MODULE_GLOBALS(module_name)                            \
+    php_##module_name##_globals module_name##_globals;
+#define PHP_EXTERN_MODULE_GLOBALS(module_name)                             \
+    extern php_##module_name##_globals module_name##_globals;
+#define PHP_INIT_MODULE_GLOBALS(module_name, globals_ctor, globals_dtor)   \
+    globals_ctor(&module_name##_globals);
+
+#endif
 
 /* global variables */
 extern pval *data;