kylefang/react-native
1
0
Fork 0
mirror of https://github.com/zhigang1992/react-native.git synced 2026-01-12 22:50:10 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fabric: Fixed memory leak happened between RCTSurfacePresenter and RCTBridge

Browse Source 
Summary: RCTBridge does not need to retain RCTSurfacePresenter, so we enforce that using `OBJC_ASSOCIATION_ASSIGN`.

Reviewed By: mdvacca

Differential Revision: D15273325

fbshipit-source-id: f223192ff5f781d9e905b004907739a36882bb63
This commit is contained in:
Valentin Shergin
2019-05-09 10:46:03 -07:00
committed by Facebook Github Bot
parent 4e215b20a3
commit c60d9f8c15
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
React/Fabric/RCTSurfacePresenter.mm
View File

@@ -410,7 +410,7 @@ using namespace facebook::react;
- (void)setSurfacePresenter:(RCTSurfacePresenter *)surfacePresenter
{
objc_setAssociatedObject(self, @selector(surfacePresenter), surfacePresenter, OBJC_ASSOCIATION_RETAIN_NONATOMIC);
objc_setAssociatedObject(self, @selector(surfacePresenter), surfacePresenter, OBJC_ASSOCIATION_ASSIGN);
}
- (RCTSurfacePresenter *)surfacePresenter