Marc Rousavy

Results 1816 comments of Marc Rousavy

Hey I think this is fixed in the latest version of Visioncamera.

@javache what do you think of putting it inside `ReactInstanceManager::tearDownReactContext`?

Or I guess at the end of `ReactInstanceManager::recreateReactContextInBackground` - after that ran, `this.mCurrentReactContext` is also null, and it definitely looks like this is a better place for a "GC when...

Okay I moved it to `recreateReactContextInBackground` and added a comment 👍 let me know if that makes sense EDIT: hmm actually I don't think this makes sense - for bridge...

I just moved it into `DevSupportManagerBase.java::onReactInstanceDestroyed(...)`, what do you think about that? For me this always hits after refreshing in debug. In release, this isn't used afaik

sorry for the bump here @javache - have you had a chance to look at this?

This is dope! Nice work @piaskowyk 🔥

Good feature request 👍 This should be simple to add in the native code, unfortunately I don't have any free time for such enhancements right now.