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

completeTransactions leak #110

Closed
3 of 9 tasks
aguila9 opened this issue Nov 21, 2016 · 4 comments
Closed
3 of 9 tasks

completeTransactions leak #110

aguila9 opened this issue Nov 21, 2016 · 4 comments

Comments

@aguila9
Copy link

aguila9 commented Nov 21, 2016

Platform

  • iOS
  • macOS
  • tvOS

In app purchase type

  • Consumable
  • Non-consumable
  • Auto-Renewable Subscription
  • Non-Renewing Subscription

Environment

  • Sandbox
  • Production

Version

0.6

Report

Issue summary

Hi, thanks for your great work.

I implemented "completeIAPTransactions()" inside the AppDelegate, as you suggest. I'm using the same code that you use in your demo app. Instruments is showing a small leak.

_NativeDictionaryStorageImpl<String, InAppProductPurchaseRequest>
_NativeDictionaryStorageOwner<String, InAppProductPurchaseRequest>

chordleak

Are you aware of this? or is the problem on my side?

Thanks for your help.

@delucamike
Copy link

delucamike commented Dec 19, 2016

I am experiencing the same leak. I checked the demo app and that seems to have the leak as well.

@bizz84
Copy link
Owner

bizz84 commented Jan 21, 2017

InAppProductPurchaseRequest will be removed here: #131

Once the PR is merged, would be good to test for similar leaks.

@bizz84
Copy link
Owner

bizz84 commented Feb 4, 2017

@aguila9 could you retest this on version 0.7.0? If this issue is not updated in a few weeks I will close it.

@bizz84
Copy link
Owner

bizz84 commented Feb 20, 2017

Tracking memory leaks on this other issue: #120

The one you refer to should no longer happen as InAppProductPurchaseRequest was removed on version 0.7.0.

Closing this issue.

@bizz84 bizz84 closed this as completed Feb 20, 2017
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

No branches or pull requests

3 participants