太占内存了，4G内存完全不够占的，使用了opencc_close

nauxliu / opencc4php

简繁体转换 PHP 扩展

MIT License

458 stars 85 forks source link

太占内存了，4G内存完全不够占的，使用了opencc_close #15

Closed tooktoo closed 8 years ago

tooktoo commented 8 years ago

$od = opencc_open("s2twp.json"); opencc_close($od); 就是不释放内存~！

pkujhd commented 8 years ago

@tooktoo 代码在104行(如果你使用的是php7.0+）有memleak，将这个语句Z_TYPE_INFO_P(zod) = IS_NULL;换成zend_list_close(Z_RES_P(zod)); 这个语句会造成56bytes的泄露(php7)，如果是php5的话，修改102行为 zend_list_delete(Z_RESVAL_P(zod)); @NauxLiu， zod是php层传递进来的变量，这个的析构应该是zend负责处理的，不需要处理成IS_NULL的类型. 我参考了ext/finfo的这个系统扩展的处理方式

nauxliu commented 8 years ago

已修复： #17

tooktoo commented 8 years ago

@pkujhd 感谢，能把你邮件发给我吗？我还想向你咨询点问题。谢谢

pkujhd commented 8 years ago

@tooktoo , 我的email是pkujhd@gmail.com