Skip to content

Commit

Permalink
Remove mTurboModuleManagerJSIModule from CatalystInstanceImpl
Browse files Browse the repository at this point in the history
Summary:
I'm removing mTurboModuleManagerJSIModule from CatalystInstanceImpl, as this is a duplicated variable.

changelog: [internal] internal

Differential Revision: D49483637

fbshipit-source-id: 5544e3a5431e669dcac52afccd9a3f2277cea36c
  • Loading branch information
mdvacca authored and facebook-github-bot committed Sep 26, 2023
1 parent 13282c7 commit c4b8944
Showing 1 changed file with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,6 @@ public String toString() {

private JavaScriptContextHolder mJavaScriptContextHolder;
private volatile @Nullable TurboModuleRegistry mTurboModuleRegistry = null;
private @Nullable JSIModule mTurboModuleManagerJSIModule = null;

// C++ parts
private final HybridData mHybridData;
Expand Down Expand Up @@ -369,8 +368,8 @@ public void run() {
@Override
public void run() {
// We need to destroy the TurboModuleManager on the JS Thread
if (mTurboModuleManagerJSIModule != null) {
mTurboModuleManagerJSIModule.invalidate();
if (mTurboModuleRegistry != null) {
mTurboModuleRegistry.invalidate();
}

getReactQueueConfiguration()
Expand Down Expand Up @@ -586,7 +585,6 @@ public void run() {

public void setTurboModuleManager(JSIModule module) {
mTurboModuleRegistry = (TurboModuleRegistry) module;
mTurboModuleManagerJSIModule = module;
}

private void decrementPendingJSCalls() {
Expand Down

0 comments on commit c4b8944

Please sign in to comment.