Skip to content

Commit

Permalink
Fix #23755 ("RCTImagePickerManager requires main queue setup" warning) (
Browse files Browse the repository at this point in the history
#24314)

Summary:
Fix #23755 - Which is to remove the warning:
```
Module RCTImagePickerManager requires main queue setup since it overrides `init` but doesn't implement `requiresMainQueueSetup`. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.
```

General Fixed - Warning "RCTImagePickerManager requires main queue setup
Pull Request resolved: #24314

Differential Revision: D14788772

Pulled By: cpojer

fbshipit-source-id: e2017136008367d36468debb258afa698b5402bc
  • Loading branch information
Seph Soliman authored and facebook-github-bot committed Apr 4, 2019
1 parent 6ca438a commit 6508b88
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion Libraries/CameraRoll/RCTImagePickerManager.m
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,14 @@ - (id)init
return self;
}

+ (BOOL)requiresMainQueueSetup
{
return NO;
}

- (void)dealloc
{
[[NSNotificationCenter defaultCenter] removeObserver:self name:@"AVCaptureDeviceDidStartRunningNotification" object:nil];
[[NSNotificationCenter defaultCenter] removeObserver:self name:@"AVCaptureDeviceDidStartRunningNotification" object:nil];
}

- (dispatch_queue_t)methodQueue
Expand Down

0 comments on commit 6508b88

Please sign in to comment.