I’m pretty sure the example code has a memory cycle. The view controller owns the table view, which owns the table view cells, which have a buttonAction closure that strongly captures the view controller. The closure should use weak or unowned to avoid capturing self.