KeyboardKit: Memory leak?
After fixing a memory leak involved with gesture applying extensions in #72 there are still one or more potential memory leaks, as described in the ticket:
“In the beginning, the memory consumption is about 30 MB, but after changing the keyboard type several times, especially after scrolled through emojis in the collection view, the memory consumption keeps increasing significantly. In the end, it reached 103.2 MB.”
For me, in a simulator, the memory consumption is stable and I do not see any huge increases in memory consumption as I change keyboards both globally and within the demo.
I have however adjusted the code in how StandardKeyboardActionHandler returns actions, where I have added weak self. I see no difference before and after doing this, but still think it’s correct to have weak self there.
You can try these changes in the memory-leaks branch.
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Comments: 16
Hi @danielsaidi, will check it later in my spare time