Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

一些bug和注释的修正 #30

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

一些bug和注释的修正 #30

wants to merge 11 commits into from

Conversation

koboshi
Copy link

@koboshi koboshi commented Jan 3, 2014

主要包括:
ezip函数bug修正
is_utf8函数性能改进
动态调用函数改为用Reflection类实现
Redis驱动类Delete方法根据手册 (https://github.com/owlient/phpredis) 对返回值进行修改
一些轻微的语法错误修改 (new Obj; --> new Obj();)
一些函数和类方法注释不正确和不完整进行添加补充修改

@duguying
Copy link
Contributor

关于ezip:
在php5.3环境下成功解压出来240K的文件包,还有就是你对第550和553行的删除与修改导致解压出来的文件零散的放置于当前文件夹而不是放置在单独以文件名(不包括扩展名)为文件夹名的文件夹内,这样很凌乱,作为原创者我驳回你对第550和553行的修改。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants