Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RCTVideo initialized twice? #29

Closed
qingfeng opened this issue May 5, 2015 · 5 comments
Closed

RCTVideo initialized twice? #29

qingfeng opened this issue May 5, 2015 · 5 comments

Comments

@qingfeng
Copy link
Contributor

qingfeng commented May 5, 2015

I added a log:

- (UIView *)view
{
  NSLog(@"RCTVideo init RCTVideo init");
  return [[RCTVideo alloc] initWithEventDispatcher:self.bridge.eventDispatcher];
}

result:

2015-05-06 02:50:18.218 gouhuo-native[67470:4294635] RCTVideo init RCTVideo init
2015-05-06 02:50:18.218 gouhuo-native[67470:4294635] RCTVideo init RCTVideo init

This is probably what is the problem? thanks

@brentvatne
Copy link
Contributor

Hi @qingfeng - if you add a breakpoint in there and inspect the stack you will see that one of them is being created for purposes internal to React Native, in fact this happens with all components 😄

@qingfeng
Copy link
Contributor Author

Really it is. . .
Is react-native problem, but, react-native can load ViewController only one more time?
Because, I encountered a problem, such as a microphone loaded twice if there is a problem

@qingfeng
Copy link
Contributor Author

thanks! 🍺

@brentvatne
Copy link
Contributor

😸 no problem @qingfeng

hueniverse pushed a commit that referenced this issue Jun 9, 2022
This PR will fix the crash from the experiment on react-native-video

Jira: VEX-7093

velocity PR: crunchyroll/velocity#2443

The crash was caused by using the player object before it was set by another thread, the change simply waits for the player to be available before executing any player interaction after DRM initialization.

Reviews
Major reviewer (domain expert): @jctorresM
Minor reviewer: @jacob-livingston
yungblud pushed a commit to react-native-yeouido/react-native-video that referenced this issue Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants