#46: fix shared issue

Author: EdmondDantes

async.c

@@ -29,9 +29,7 @@
 #include "async_API.h"
 #include "async_arginfo.h"
 #include "zend_interfaces.h"
-#ifdef PHP_ASYNC_LIBUV
 #include "libuv_reactor.h"
-#endif
 
 zend_class_entry *async_ce_awaitable = NULL;
 zend_class_entry *async_ce_timeout = NULL;
@@ -889,8 +887,6 @@ static PHP_GINIT_FUNCTION(async)
 #endif
 
 	async_globals->reactor_started = false;
-
-#ifdef PHP_ASYNC_LIBUV
 	async_globals->signal_handlers = NULL;
 	async_globals->signal_events = NULL;
 	async_globals->process_events = NULL;
@@ -906,7 +902,6 @@ static PHP_GINIT_FUNCTION(async)
 	async_globals->uvloop_wakeup = NULL;
 	async_globals->pid_queue = NULL;
 #endif
-#endif
 }
 
 /* {{{ PHP_GSHUTDOWN_FUNCTION */
@@ -932,23 +927,16 @@ ZEND_MINIT_FUNCTION(async)
 	// async_register_future_ce();
 
 	async_scheduler_startup();
-
 	async_api_register();
-
-#ifdef PHP_ASYNC_LIBUV
 	async_libuv_reactor_register();
-#endif
 
 	return SUCCESS;
 }
 
 ZEND_MSHUTDOWN_FUNCTION(async)
 {
 	// async_scheduler_shutdown();
-
-#ifdef PHP_ASYNC_LIBUV
 	// async_libuv_shutdown();
-#endif
 
 	return SUCCESS;
 }
@@ -959,11 +947,7 @@ PHP_MINFO_FUNCTION(async)
 	php_info_print_table_header(2, "Module", PHP_ASYNC_NAME);
 	php_info_print_table_row(2, "Version", PHP_ASYNC_VERSION);
 	php_info_print_table_row(2, "Support", "Enabled");
-#ifdef PHP_ASYNC_LIBUV
 	php_info_print_table_row(2, "LibUv Reactor", "Enabled");
-#else
-	php_info_print_table_row(2, "LibUv Reactor", "Disabled");
-#endif
 	php_info_print_table_end();
 }
 

config.m4

@@ -32,7 +32,6 @@ if test "$PHP_ASYNC" = "yes"; then
         LIBUV_LIBLINE=`$PKG_CONFIG libuv --libs`
         LIBUV_VERSION=`$PKG_CONFIG libuv --modversion`
         AC_MSG_RESULT(from pkgconfig: found version $LIBUV_VERSION)
-        AC_DEFINE(PHP_ASYNC_LIBUV,1,[ ])
       else
         AC_MSG_ERROR(system libuv must be upgraded to version >= 1.44.0 (fixes UV_RUN_ONCE busy loop issue))
       fi
@@ -57,7 +56,6 @@ if test "$PHP_ASYNC" = "yes"; then
       PHP_CHECK_LIBRARY(uv, uv_version,
       [
         PHP_ADD_LIBRARY_WITH_PATH(uv, $UV_DIR/$PHP_LIBDIR, UV_SHARED_LIBADD)
-        AC_DEFINE(PHP_ASYNC_LIBUV,1,[ ])
       ],[
         AC_MSG_ERROR([wrong uv library version or library not found])
       ],[

config.w32

@@ -4,16 +4,7 @@ ARG_ENABLE('async', 'Enable True Async', 'no');
 
 if (PHP_ASYNC == "yes") {
 
-    if(typeof PHP_ASYNC_API == "undefined" && typeof PHP_EXPERIMENTAL_ASYNC_API != "undefined") {
-        var PHP_ASYNC_API = PHP_EXPERIMENTAL_ASYNC_API;
-    }
-
-    if (!PHP_ASYNC_API || PHP_ASYNC_API != "yes") {
-        ERROR("PHP TRUE ASYNC API is required. Please configure PHP with --with-async-api.");
-    }
-
-    EXTENSION("async", "async.c coroutine.c scope.c scheduler.c exceptions.c iterator.c async_API.c");
-    ADD_SOURCES("ext/async","zend_common.c context.c");
+    EXTENSION("async", "async.c coroutine.c scope.c scheduler.c exceptions.c iterator.c async_API.c zend_common.c context.c libuv_reactor.c");
     ADD_SOURCES("ext/async/internal", "allocator.c circular_buffer.c");
 
     ADD_FLAG("CFLAGS", "/D PHP_ASYNC");
@@ -27,17 +18,14 @@ if (PHP_ASYNC == "yes") {
     PHP_INSTALL_HEADERS("ext/async", "iterator.h");
     PHP_INSTALL_HEADERS("ext/async", "async_API.h");
     PHP_INSTALL_HEADERS("ext/async", "context.h");
+    PHP_INSTALL_HEADERS("ext/async", "libuv_reactor.h");
 
     if (CHECK_HEADER_ADD_INCLUDE("libuv/uv.h", "CFLAGS_UV", PHP_PHP_BUILD + "\\include")
         && CHECK_LIB("libuv.lib", "libuv")) {
 
         // Note: libuv >= 1.44.0 is required for UV_RUN_ONCE busy loop fix
         // For Windows builds, manually verify libuv version meets requirements
-        
-        PHP_INSTALL_HEADERS("ext/async", "libuv_reactor.h");
 
-        ADD_SOURCES("ext/async", "libuv_reactor.c");
-        ADD_FLAG("CFLAGS", "/D PHP_ASYNC_LIBUV");
         ADD_FLAG("LIBS", "libuv.lib Dbghelp.lib Userenv.lib");
     } else {
         ERROR("Libuv components are not found. The search was performed in the directory: '" + PHP_PHP_BUILD +

coroutine.h

@@ -16,7 +16,7 @@
 #ifndef COROUTINE_H
 #define COROUTINE_H
 
-#include "php_async.h"
+#include "php_async_api.h"
 #include <Zend/zend_async_API.h>
 
 ZEND_STACK_ALIGNED void async_coroutine_execute(zend_fiber_transfer *transfer);

exceptions.c

@@ -321,7 +321,7 @@ void async_apply_exception(zend_object **to_exception)
 	}
 }
 
-void async_rethrow_exception(zend_object *exception)
+PHP_ASYNC_API void async_rethrow_exception(zend_object *exception)
 {
 	if (EG(current_execute_data)) {
 		zend_throw_exception_internal(exception);

exceptions.h

@@ -19,6 +19,7 @@
 #include <zend_portability.h>
 #include <zend_property_hooks.h>
 #include <Zend/zend_async_API.h>
+#include "php_async_api.h"
 
 BEGIN_EXTERN_C()
 
@@ -42,7 +43,7 @@ PHP_ASYNC_API void async_composite_exception_add_exception(zend_object *composit
 bool async_spawn_and_throw(zend_object *exception, zend_async_scope_t *scope, int32_t priority);
 void async_apply_exception_to_context(zend_object *exception);
 zend_object *async_extract_exception(void);
-void async_rethrow_exception(zend_object *exception);
+PHP_ASYNC_API void async_rethrow_exception(zend_object *exception);
 void async_apply_exception(zend_object **to_exception);
 
 END_EXTERN_C()

php_async.h

@@ -18,13 +18,11 @@
 
 #include <php.h>
 
-#ifdef PHP_ASYNC_LIBUV
 #ifdef PHP_WIN32
 #include "libuv/uv.h"
 #else
 #include <uv.h>
 #endif
-#endif
 
 #include "coroutine.h"
 #include "internal/circular_buffer.h"
@@ -38,26 +36,14 @@
 extern zend_module_entry async_module_entry;
 #define phpext_async_ptr &async_module_entry
 
-#ifdef PHP_WIN32
-# ifdef ASYNC_EXPORTS
-#  define PHP_ASYNC_API __declspec(dllexport)
-# else
-#  define PHP_ASYNC_API __declspec(dllimport)
-# endif
-#else
-# if defined(__GNUC__) && __GNUC__ >= 4
-#  define PHP_ASYNC_API __attribute__ ((visibility("default")))
-# else
-#  define PHP_ASYNC_API
-# endif
-#endif
+#include "php_async_api.h"
 
 PHP_ASYNC_API extern zend_class_entry *async_ce_awaitable;
 PHP_ASYNC_API extern zend_class_entry *async_ce_timeout;
 
-#define PHP_ASYNC_NAME "TrueAsync"
-#define PHP_ASYNC_VERSION "0.5.0"
-#define PHP_ASYNC_NAME_VERSION "TrueAsync v0.5.0"
+#define PHP_ASYNC_NAME "true_async"
+#define PHP_ASYNC_VERSION "0.4.0"
+#define PHP_ASYNC_NAME_VERSION "true async v0.4.0"
 
 typedef struct
 {
@@ -96,7 +82,6 @@ zend_async_context_t *root_context;
 /* The default concurrency */
 int default_concurrency;
 
-#ifdef PHP_ASYNC_LIBUV
 /* The reactor */
 uv_loop_t uvloop;
 bool reactor_started;
@@ -115,7 +100,6 @@ uv_async_t *uvloop_wakeup;
 /* Circular buffer of libuv_process_t ptr */
 circular_buffer_t *pid_queue;
 #endif
-#endif
 
 #ifdef PHP_WIN32
 #endif

php_async_api.h

@@ -0,0 +1,33 @@
+/*
++----------------------------------------------------------------------+
+  | Copyright (c) The PHP Group                                          |
+  +----------------------------------------------------------------------+
+  | This source file is subject to version 3.01 of the PHP license,      |
+  | that is bundled with this package in the file LICENSE, and is        |
+  | available through the world-wide-web at the following url:           |
+  | https://www.php.net/license/3_01.txt                                 |
+  | If you did not receive a copy of the PHP license and are unable to   |
+  | obtain it through the world-wide-web, please send a note to          |
+  | license@php.net so we can mail you a copy immediately.               |
+  +----------------------------------------------------------------------+
+  | Author: Edmond                                                       |
+  +----------------------------------------------------------------------+
+*/
+#ifndef PHP_ASYNC_API_H
+#define PHP_ASYNC_API_H
+
+#ifdef PHP_WIN32
+# ifdef ASYNC_EXPORTS
+#  define PHP_ASYNC_API __declspec(dllexport)
+# else
+#  define PHP_ASYNC_API __declspec(dllimport)
+# endif
+#else
+# if defined(__GNUC__) && __GNUC__ >= 4
+#  define PHP_ASYNC_API __attribute__ ((visibility("default")))
+# else
+#  define PHP_ASYNC_API
+# endif
+#endif
+
+#endif // PHP_ASYNC_API_H