remove useless old php logics

Author: doubaokun

ext-src/php_swoole.cc

@@ -160,26 +160,11 @@ static void php_swoole_init_globals(zend_openswoole_globals *openswoole_globals)
 
 void php_swoole_register_shutdown_function(const char *function) {
     php_shutdown_function_entry shutdown_function_entry;
-#if PHP_VERSION_ID >= 80100
     zval function_name;
     ZVAL_STRING(&function_name, function);
     zend_fcall_info_init(
         &function_name, 0, &shutdown_function_entry.fci, &shutdown_function_entry.fci_cache, NULL, NULL);
     register_user_shutdown_function(Z_STRVAL(function_name), Z_STRLEN(function_name), &shutdown_function_entry);
-#else
-    zval *function_name;
-#if PHP_VERSION_ID >= 80000
-    shutdown_function_entry.arg_count = 0;
-    shutdown_function_entry.arguments = NULL;
-    function_name = &shutdown_function_entry.function_name;
-#else
-    shutdown_function_entry.arg_count = 1;
-    shutdown_function_entry.arguments = (zval *) safe_emalloc(sizeof(zval), 1, 0);
-    function_name = &shutdown_function_entry.arguments[0];
-#endif
-    ZVAL_STRING(function_name, function);
-    register_user_shutdown_function(Z_STRVAL_P(function_name), Z_STRLEN_P(function_name), &shutdown_function_entry);
-#endif
 }
 
 void php_swoole_set_global_option(HashTable *vht) {
@@ -1198,7 +1183,7 @@ PHP_MINIT_FUNCTION(openswoole) {
     SwooleG.dns_cache_refresh_time = 60;
 
     // enable pcre.jit and use swoole extension on MacOS will lead to coredump, disable it temporarily
-#if defined(PHP_PCRE_VERSION) && defined(HAVE_PCRE_JIT_SUPPORT) && PHP_VERSION_ID >= 70300 && __MACH__ &&              \
+#if defined(PHP_PCRE_VERSION) && defined(HAVE_PCRE_JIT_SUPPORT) && __MACH__ &&              \
     !defined(SW_DEBUG)
     PCRE_G(jit) = 0;
 #endif

ext-src/php_swoole_coroutine.h

@@ -53,9 +53,7 @@ struct PHPContext {
     zend_vm_stack vm_stack;
     size_t vm_stack_page_size;
     zend_execute_data *execute_data;
-#if PHP_VERSION_ID >= 80000
     uint32_t jit_trace_num;
-#endif
 #ifdef ZEND_CHECK_STACK_LIMIT
     void *stack_base;
     void *stack_limit;
@@ -64,10 +62,6 @@ struct PHPContext {
     zend_class_entry *exception_class;
     zend_object *exception;
     zend_output_globals *output_ptr;
-#if PHP_VERSION_ID < 80100
-    /* for array_walk non-reentrancy */
-    zend::Function *array_walk_fci;
-#endif
     /* for error control `@` */
     bool in_silence;
     bool enable_scheduler;

ext-src/php_swoole_cxx.cc

@@ -12,24 +12,8 @@ SW_API zend_string **sw_zend_known_strings = nullptr;
 
 //----------------------------------Swoole known string------------------------------------
 
-#if PHP_VERSION_ID < 80000
-typedef zval zend_source_string_t;
-#else
 typedef zend_string zend_source_string_t;
-#endif
 
-#if PHP_VERSION_ID < 80200
-static zend_op_array *swoole_compile_string(zend_source_string_t *source_string, ZEND_STR_CONST char *filename);
-
-// for compatibly with dis_eval
-static zend_op_array *(*old_compile_string)(zend_source_string_t *source_string, ZEND_STR_CONST char *filename);
-
-static zend_op_array *swoole_compile_string(zend_source_string_t *source_string, ZEND_STR_CONST char *filename) {
-    zend_op_array *opa = old_compile_string(source_string, filename);
-    opa->type = ZEND_USER_FUNCTION;
-    return opa;
-}
-#else
 static zend_op_array *swoole_compile_string(zend_source_string_t *source_string,
                                             ZEND_STR_CONST char *filename,
                                             zend_compile_position position);
@@ -45,7 +29,6 @@ static zend_op_array *swoole_compile_string(zend_source_string_t *source_string,
     opa->type = ZEND_USER_FUNCTION;
     return opa;
 }
-#endif
 
 namespace zend {
 bool eval(const std::string &code, std::string const &filename) {

ext-src/php_swoole_cxx.h

@@ -443,11 +443,7 @@ static inline void assign_zend_string_by_val(zval *zdata, char *addr, size_t len
     ZVAL_STR(zdata, zstr);
 }
 
-#if PHP_VERSION_ID < 80000
-#define ZEND_STR_CONST
-#else
 #define ZEND_STR_CONST const
-#endif
 
 #ifndef ZEND_BEGIN_ARG_WITH_TENTATIVE_RETURN_TYPE_INFO_EX
 #define ZEND_BEGIN_ARG_WITH_TENTATIVE_RETURN_TYPE_INFO_EX(name, return_reference, required_num_args, type, allow_null) \

ext-src/php_swoole_private.h

@@ -98,8 +98,8 @@ extern PHPAPI int php_array_merge(zend_array *dest, zend_array *src);
 #define SWOOLE_SOCKETS_SUPPORT
 #endif
 
-#if PHP_VERSION_ID < 70200
-#error "require PHP version 7.2 or later"
+#if PHP_VERSION_ID < 80200
+#error "require PHP version 8.2 or later"
 #endif
 
 #if defined(ZTS) && defined(SW_USE_THREAD_CONTEXT)
@@ -377,22 +377,12 @@ zend_bool php_swoole_signal_isset_handler(int signo);
 /*}}}*/
 
 /* PHP 8 compatibility macro {{{*/
-#if PHP_VERSION_ID < 80000
-#define sw_zend7_object zval
-#define SW_Z7_OBJ_P(object) Z_OBJ_P(object)
-#define SW_Z8_OBJ_P(zobj) zobj
-#else
 #define sw_zend7_object zend_object
 #define SW_Z7_OBJ_P(object) object
 #define SW_Z8_OBJ_P(zobj) Z_OBJ_P(zobj)
-#endif
 /*}}}*/
 
-#if PHP_VERSION_ID < 70400
-typedef size_t php_stream_size_t;
-#else
 typedef ssize_t php_stream_size_t;
-#endif
 
 /* PHP 7 wrapper functions / macros */
 
@@ -406,18 +396,8 @@ typedef ssize_t php_stream_size_t;
         (ptr) = &(val);                                                                                                \
     } while (0)
 
-#if PHP_VERSION_ID < 80000
-#define SW_ZVAL_SOCKET(return_value, result)                                                                           \
-    ZVAL_RES(return_value, zend_register_resource((void *) (result), php_sockets_le_socket()))
-#else
 #define SW_ZVAL_SOCKET(return_value, result) ZVAL_OBJ(return_value, &result->std)
-#endif
-
-#if PHP_VERSION_ID < 80000
-#define SW_Z_SOCKET_P(zsocket) (php_socket *) zend_fetch_resource_ex(zsocket, nullptr, php_sockets_le_socket())
-#else
 #define SW_Z_SOCKET_P(zsocket) Z_SOCKET_P(zsocket)
-#endif
 
 #ifndef ZVAL_IS_BOOL
 static sw_inline zend_bool ZVAL_IS_BOOL(zval *v) {
@@ -613,13 +593,7 @@ static sw_inline void add_assoc_ulong_safe(zval *arg, const char *key, zend_ulon
         }                                                                                                              \
     } while (0)
 
-#if PHP_VERSION_ID < 80100
-#define SW_SET_CLASS_NOT_SERIALIZABLE(module)                                                                          \
-    module##_ce->serialize = zend_class_serialize_deny;                                                                \
-    module##_ce->unserialize = zend_class_unserialize_deny;
-#else
 #define SW_SET_CLASS_NOT_SERIALIZABLE(module) module##_ce->ce_flags |= ZEND_ACC_NOT_SERIALIZABLE;
-#endif
 
 #define sw_zend_class_clone_deny NULL
 #define SW_SET_CLASS_CLONEABLE(module, _clone_obj) module##_handlers.clone_obj = _clone_obj
@@ -690,25 +664,9 @@ static sw_inline int sw_zend_register_class_alias(const char *name, size_t name_
 
     zend_string *_interned_name = zend_new_interned_string(_name);
 
-#if PHP_VERSION_ID >= 70300
-    return zend_register_class_alias_ex(ZSTR_VAL(_interned_name), ZSTR_LEN(_interned_name), ce, 1);
-#else
     return zend_register_class_alias_ex(ZSTR_VAL(_interned_name), ZSTR_LEN(_interned_name), ce);
-#endif
 }
 
-#if PHP_VERSION_ID < 70300
-/* Allocates object type and zeros it, but not the properties.
- * Properties MUST be initialized using object_properties_init(). */
-static zend_always_inline void *zend_object_alloc(size_t obj_size, zend_class_entry *ce) {
-    void *obj = emalloc(obj_size + zend_object_properties_size(ce));
-    /* Subtraction of sizeof(zval) is necessary, because zend_object_properties_size() may be
-     * -sizeof(zval), if the object has no properties. */
-    memset(obj, 0, obj_size - sizeof(zval));
-    return obj;
-}
-#endif
-
 static sw_inline zend_object *sw_zend_create_object(zend_class_entry *ce, zend_object_handlers *handlers) {
     zend_object *object = (zend_object *) zend_object_alloc(sizeof(zend_object), ce);
     zend_object_std_init(object, ce);
@@ -733,21 +691,6 @@ static sw_inline zend_object *sw_zend_create_object_deny(zend_class_entry *ce) {
     return object;
 }
 
-#if PHP_VERSION_ID < 80000
-static sw_inline void sw_zend_class_unset_property_deny(zval *zobject, zval *zmember, void **cache_slot) {
-    zend_class_entry *ce = Z_OBJCE_P(zobject);
-    while (ce->parent) {
-        ce = ce->parent;
-    }
-    SW_ASSERT(ce->type == ZEND_INTERNAL_CLASS);
-    if (EXPECTED(zend_hash_find(&ce->properties_info, Z_STR_P(zmember)))) {
-        zend_throw_error(
-            NULL, "Property %s of class %s cannot be unset", Z_STRVAL_P(zmember), SW_Z_OBJCE_NAME_VAL_P(zobject));
-        return;
-    }
-    std_object_handlers.unset_property(zobject, zmember, cache_slot);
-}
-#else
 static sw_inline void sw_zend_class_unset_property_deny(zend_object *object, zend_string *member, void **cache_slot) {
     zend_class_entry *ce = object->ce;
     while (ce->parent) {
@@ -760,7 +703,6 @@ static sw_inline void sw_zend_class_unset_property_deny(zend_object *object, zen
     }
     std_object_handlers.unset_property(object, member, cache_slot);
 }
-#endif
 
 static sw_inline zval *sw_zend_read_property(zend_class_entry *ce, zval *obj, const char *s, int len, int silent) {
     zval rv, *property = zend_read_property(ce, SW_Z8_OBJ_P(obj), s, len, silent, &rv);
@@ -873,12 +815,8 @@ static sw_inline zend_bool sw_zend_is_callable_at_frame(zval *zcallable,
                                                         char **error) {
     zend_string *name;
     zend_bool ret;
-#if PHP_VERSION_ID < 80000
-    ret = zend_is_callable_ex(zcallable, zobject ? Z_OBJ_P(zobject) : NULL, check_flags, &name, fci_cache, error);
-#else
     ret = zend_is_callable_at_frame(zcallable, zobject ? Z_OBJ_P(zobject) : NULL, frame, check_flags, fci_cache, error);
     name = zend_get_callable_name_ex(zcallable, zobject ? Z_OBJ_P(zobject) : NULL);
-#endif
     if (callable_name) {
         *callable_name = estrndup(ZSTR_VAL(name), ZSTR_LEN(name));
     }
@@ -921,11 +859,7 @@ static sw_inline int sw_zend_call_function_ex(
     fci.retval = retval ? retval : &_retval;
     fci.param_count = param_count;
     fci.params = params;
-#if PHP_VERSION_ID >= 80000
     fci.named_params = NULL;
-#else
-    fci.no_separation = 0;
-#endif
 
     ret = zend_call_function(&fci, fci_cache);
 
@@ -1038,10 +972,7 @@ static sw_inline char *php_swoole_url_encode(const char *value, size_t value_len
 }
 
 static sw_inline char *php_swoole_http_build_query(zval *zdata, size_t *length, smart_str *formstr) {
-#if PHP_VERSION_ID < 80000
-    if (php_url_encode_hash_ex(
-            HASH_OF(zdata), formstr, NULL, 0, NULL, 0, NULL, 0, NULL, NULL, (int) PHP_QUERY_RFC1738) == FAILURE) {
-#elif PHP_VERSION_ID < 80300
+#if PHP_VERSION_ID < 80300
     if (HASH_OF(zdata)) {
         php_url_encode_hash_ex(HASH_OF(zdata), formstr, NULL, 0, NULL, 0, NULL, 0, NULL, NULL, (int) PHP_QUERY_RFC1738);
     } else {

ext-src/swoole_atomic.cc

@@ -17,11 +17,7 @@
 #include "php_swoole_private.h"
 #include "swoole_memory.h"
 
-#if PHP_VERSION_ID >= 80000
 #include "swoole_atomic_arginfo.h"
-#else
-#include "swoole_atomic_arginfo_legacy.h"
-#endif
 
 #ifdef HAVE_FUTEX
 #include <linux/futex.h>

ext-src/swoole_atomic_arginfo_legacy.h

@@ -21,11 +21,7 @@
 
 #include "swoole_coroutine_channel.h"
 
-#if PHP_VERSION_ID >= 80000
 #include "swoole_channel_coro_arginfo.h"
-#else
-#include "swoole_channel_coro_arginfo_legacy.h"
-#endif
 
 using swoole::coroutine::Channel;
 

ext-src/swoole_channel_coro.cc

@@ -18,11 +18,7 @@
 #include "php_swoole_client.h"
 #include "swoole_mqtt.h"
 
-#if PHP_VERSION_ID >= 80000
 #include "swoole_client_arginfo.h"
-#else
-#include "swoole_client_arginfo_legacy.h"
-#endif
 
 #include <string>
 #include <queue>