Skip to content

(+): enabled ARC memory management. #1

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

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

Conversation

Panajev
Copy link

@Panajev Panajev commented Sep 10, 2012

I used Cocos2D as an external static library to test this.

Instructions on how to obtain the complete library package (CocosLib +
CocosTools + Utilities), build it, and install it can be found here:

https://github.com/Panajev/CocosLib#howto-install

I used Cocos2D as an external static library to test this.

Instructions on how to obtain the complete library package (CocosLib +
CocosTools + Utilities), build it, and install it can be found here:

https://github.com/Panajev/CocosLib#howto-install
@gregharding
Copy link
Owner

hi Panajev,

thanks for the update. I don't know anything about ARC yet so I'm not sure whether to merge the changes or not. Perhaps they could be conditionally compiled or something?

@Panajev
Copy link
Author

Panajev commented Oct 3, 2012

Hey, sorry for taking so long to answer back (btw, hoping CCRenderTexture behaves better now :)).

Yes, conditional compilation is indeed possible. I'll review the changes I made and conditionally enable ARC.

Brought back old funcgtionality when ARC is disabled.
@Panajev
Copy link
Author

Panajev commented Oct 3, 2012

Now, it should be easier to merge in I think.

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