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

修改wax的pod spec,增加xml2和sqlite3的支持; #11

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

Conversation

philonpang
Copy link

No description provided.

庞辉 added 8 commits November 11, 2015 14:40
(2)解决selector被其他framework替换指向_objc_forwardMsg,仍然被拦截处理的bug;
(3)增加wax_clear()进行patch现场修复;
* 'master' of https://github.com/philonpang/wax:
  修改wax的pod spec,增加xml2和sqlite3的支持;
…ation获取;

(2) 修复当覆盖oc方法中有super调用时进入死循环的错误;
…waxClass的所有selector都指向_objc_msgforward 销毁与否没有影响);

(2) wax_class 对于一个进程多次重启waxClass,仍然记录waxClass;(即使第一次没有销毁成功,当clear的时候能够再次尝试销毁)
(3) wax_instance override 方法重启,当一个method name对应class中确实有两个响应的selector方法时,override两个selector方法;
@intheway
Copy link

thanks for your PR. but there is so much code ,that i can not merge it all. I will refer some of your code. Thanks again.

@intheway
Copy link

in order to make wax more robustness, I think we need to use as less global state variable as possible. such as _lastForwardClassSelecotorKey is not recommend.

@CLAassistant
Copy link

CLAassistant commented Apr 22, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


庞辉 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

3 participants