e61a14e71d
Summary: Previously, we stored a pointer to ShadowNode inside NSAttributedString's attributes to make possible retrieving an EventEmitter associated with some text fragment. That worked fine besides only one caveat: the internal implementation of NSAttributedString is quite strange and that causes a memory leak. Because of some reason, NSAttributedString does not release stored attributes after own destruction (maybe OS uses some kind of caching). So, now, instead of storing a strong pointer to ShadowNode inside NSAttributedString, we store a weak pointer to EventEmitter. Storing a weak pointer is okay because a desired lifetime of EventEmitter is guaranteed by LocalData stored inside a View. Storing a weak EventEmitter instead of weak ShadowNode will also help us with migration to ShadowView (we cannot store ShadowView weakly because it's a stack allocated object). Reviewed By: sahrens Differential Revision: D13196886 fbshipit-source-id: f8714e4b3709765629d6456edf0c635bf5f7c53b |
||
|---|---|---|
| .. | ||
| Base | ||
| CxxBridge | ||
| CxxModule | ||
| CxxUtils | ||
| DevSupport | ||
| Fabric | ||
| Inspector | ||
| Modules | ||
| Profiler | ||
| React.xcodeproj | ||
| UIUtils | ||
| Views | ||
| .clang-format | ||
| third-party.xcconfig | ||