search

php / pecl-networking-gearman

PHP wrapper to libgearman
https://pecl.php.net/package/gearman
Other
33 stars 25 forks source link

Several fixes #4

Closed oleg-st closed 3 years ago

oleg-st commented 3 years ago

1) Reworked serialization bug fix to fix memory leaks found during tests. 2) Removed sockets extension dependency in tests 3) Minor fixes: gearman_client_clear_callbacks etc

oleg-st commented 3 years ago

Current serialization bug fix causes tests failed:

FAIL GearmanTask::functionName, gearman_task_function_name [tests/gearman_task_001.phpt] 
FAIL GearmanTask::unique, gearman_task_unique [tests/gearman_task_002.phpt] 
FAIL GearmanWorker::addFunction(), context param [tests/gearman_worker_016.phpt]

Tested on 8.0.1-dev (with debug enabled)

Output:

[Sat Jan 16 11:55:24 2021]  Script:  '/home/Oleg.Stepanischev/pecl-networking-gearman/tests/gearman_task_001.php'
/home/Oleg.Stepanischev/pecl-networking-gearman/php_gearman_task.c(21) :  Freeing 0x00007f574925e780 (112 bytes), script=/home/Oleg.Stepanischev/pecl-networking-gearman/tests/gearman_task_001.php

Moved back destructions of underlying gearman objects to __destruct