-
Notifications
You must be signed in to change notification settings - Fork 250
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
Use with ARC #2
Comments
I found the compile warning is not for ARC, they are from static analyzer, can you check this thread? Sent from my iPhone4S On 2012/06/04, at 23:24, Mark Rickertreply@reply.github.com wrote:
|
I've submitted a pull request. #3 There's not a huge issue here, it was just the static analyzer complaining about the function names that returned retained objects. For now, you can use my branch that fixes this if you'd like: markrickert/FPPopover@12a097aee05dd1980d900b345b63db3ce187d95b |
I found that you used a lot of IVAR directly, why don't you use property? |
You'd need to ask the library creator (not me). I'm guessing it's just personal coding style/preferences. |
hi @mikezang, I have used iVar directly because this is what it has to do when we don't want to expose the iVars like _window. If you use properties you will expose the ivars outside the instance. Thanks |
@mikezang : thanks to @markrickert now static analyzer giving warnings |
Thanks for your reply, Alvise. I think you can put property in implement file so that you don't need worry about expose to outside. Sent from my iPhone4S On 2012/06/08, at 0:13, Alvise Susmelreply@reply.github.com wrote:
|
How can I use your source with ARC?
The text was updated successfully, but these errors were encountered: