IQKeyboardReturnKeyHandler retain cycle

[godexsoft]

In swift code i have IQKeyboardReturnKeyHandler as per documentation. Just adding the field and creating an instance from (controller: self) makes a retain cycle somehow. Easiest way to check is to add deinit method with a log line. Without IQKeyboardReturnKeyHandler the log line appears, with IQKeyboardReturnKeyHandler it does not - the controller stays in memory.

[hackiftekhar]

I'll check the memory and retain cycle issues. Thanks for letting me know about this.

[hackiftekhar]

I tried to fix memory issues with v5.0.8, let's see if that works for you.

[godexsoft]

unfortunately it's still with the same result. deinit is not executed when creating instance from (controller: self).

[hackiftekhar]

You should release the IQReturnKeyHandler object wirh deinit of your ViewController. if it doesn't work then I'll try to take a look again.

[godexsoft]

But how can you release it in the deinit if deinit is never executed due to retain cycle? That's exactly the problem :)

What did work tho is this:

override func viewDidDisappear(_ animated: Bool) {
    super.viewDidDisappear(animated)        
    returnKeyHandler = nil
 }

deinit {
    log.debug("now called")
}

[hackiftekhar]

okay, let me see then.

[hackiftekhar]

I got the point, it's because the IQKeyboardReturnKeyHandler is keeping reference of textField delegate and most of the time it's the ViewController who owns it. A fix is on the way.