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

CCMutableArray optimization #480

Merged
merged 1 commit into from Aug 22, 2011

Conversation

moadib
Copy link
Contributor

@moadib moadib commented Aug 19, 2011

@minggo
Copy link
Contributor

minggo commented Aug 22, 2011

You can not use
std::sort(pArray->begin(), pArray->end(), less);
in
CCTouchDispatcher::rearrangeHandlers()
or it may crash. You can refer bug #680

@minggo
Copy link
Contributor

minggo commented Aug 22, 2011

Oh, you use reserve(), so I think you are right. I will check the usage of insertObjectAtIndex(), and merge your pull request. Thank you.

minggo added a commit that referenced this pull request Aug 22, 2011
@minggo minggo merged commit 8eef0bd into cocos2d:master Aug 22, 2011
dumganhar pushed a commit to dumganhar/cocos2d-x that referenced this pull request Sep 19, 2011
angeltown pushed a commit to angeltown/cocos2d-x that referenced this pull request Apr 28, 2014
angeltown pushed a commit to angeltown/cocos2d-x that referenced this pull request Apr 28, 2014
angeltown pushed a commit to angeltown/cocos2d-x that referenced this pull request Apr 29, 2014
angeltown pushed a commit to angeltown/cocos2d-x that referenced this pull request Apr 29, 2014
walzer pushed a commit that referenced this pull request May 4, 2014
minggo added a commit that referenced this pull request May 4, 2014
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.

None yet

2 participants