Retain cycle
hassila opened this issue · comments
Joakim Hassila commented
Seems #21 is back...
Joakim Hassila commented
Reproduces during flat bench lazy run.
Maxim Zaks commented
Yeah it is funny how with memory leak the Lazy run dealloc is 9 ms and without 910 ms. Makes me wonder if somehow pooling all Table classes and Lazy accessors is a good idea.
Maxim Zaks commented
Hm ... I remember now why I reintroduced the bug. if I have this
LazyVector(count: vectorLength){ [unowned self] in
FlatBuffersGeneratedAPITest
doesn't work anymore ...
Maxim Zaks commented
Ok, solved it.
Joakim Hassila commented
Looks good now.
Makes me wonder if somehow pooling all Table classes and Lazy accessors is a good idea