neilang / NAMapKit

Allows you to use custom maps in iphone applications and attempts to mimics some of the behaviour of the MapKit framework
MIT License
263 stars 52 forks source link

Retain cycle bug #31

Closed jinsongK closed 10 years ago

jinsongK commented 10 years ago

When i run demo,i found a problem. When NAPinAnnotationsDemoViewController return to the NAMasterViewController, NAPinAnnotationsDemoViewController can't release. i check this demo with instruments tool,and found NAPinAnnotationsDemoViewController' count is always increase.

dblock commented 10 years ago

Thanks for reporting this. Would love a fix, thank you.

jinsongK commented 10 years ago

Both NAPinAnnotationMapView and NACarAnnotationCallOutView, I guess.

neilang commented 10 years ago

Thanks for reporting. I briefly loaded up the latest version on the simulator with instruments. I couldn't find any memory leaks. Perhaps you could provide some more details on how to replicate the problem, or perhaps some screenshots or a video of what you are seeing? Cheers!

jinsongK commented 10 years ago
  1. Press “Pin Annotations Demo”, opens the NAPinAnnotationsDemoViewController view
  2. Exit view, returns to home screen
dblock commented 10 years ago

@jinsongK The attachments from emails don't work when you send them to github. If you can drag and drop things here that'd be great!

jinsongK commented 10 years ago
                         原始邮件                             发件人: Daniel Doubrovkine (dB.) @dblockdotorg<notifications@github.com>收件人: neilang/NAMapKit<NAMapKit@noreply.github.com>抄送: jinsongK<22339146@qq.com>发送时间: 2014年5月9日(周五) 19:43主题: Re: [NAMapKit] Retain cycle bug (#31)@jinsongK The attachments from emails don't work when you send them to github. If you can drag and drop things here that'd be great!

—Reply to this email directly or view it on GitHub.

jinsongK commented 10 years ago

step1 step2 step3