Refactor connection pooling code.

Move logic of searching+creating connection pool resource to `redis_sock_get_connection_pool`.
Don't close streams in ZEND_RSRC_DTOR_FUNC because all of them stored in `EG(persistent_list)`
and will be destroyed by PHP engine.

1 files changed, 0 insertions, 7 deletions

diff --git a/redis.c b/redis.c
index a88753f2..eaeaf1ce 100644
--- a/redis.c
+++ b/redis.c
@@ -716,16 +716,9 @@ static void add_class_constants(zend_class_entry *ce, int is_cluster TSRMLS_DC)
     zend_declare_class_constant_stringl(ce, "BEFORE", 6, "before", 6 TSRMLS_CC);
 }
 
-static void
-redis_pconnect_dtor(void *ptr TSRMLS_DC)
-{
-    php_stream_pclose(*(php_stream **)ptr);
-}
-
 static ZEND_RSRC_DTOR_FUNC(redis_connections_pool_dtor)
 {
     if (res->ptr) {
-        zend_llist_apply(res->ptr, redis_pconnect_dtor TSRMLS_CC);
         zend_llist_destroy(res->ptr);
         pefree(res->ptr, 1);
     }