-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
[Crash] Reloading with profiling enabled sends null selector to modules upon deallocation #1642
Comments
Downgrading to "react-native": "^0.5.0", everything works as expected =). Looking forward to a stable 0.6.0 and beyond. |
Hmm. I can't reproduce this. CMD + R works as expected for me on Xcode 6.3.1. It currently does nothing on Xcode 7 beta, but that's a known issue. SHIFT + CMD + Z not working is probably because the keyboard shortcut is actually CTRL + CMD + Z ;-) |
I upgraded the mod player project I shared with you guys. GitHub master is working as expected. I will create a .6.0 project from scratch and migrate things by hand. And yes, I mistyped. CTRL + CMD + Z is failing. /me slaps self |
Hi, is there any news on this topic? I get the same with 0.6.0-rc: 2015-06-24 08:00:17.307 Yoloci[15250:17707298] *** NSForwarding: warning: selector (0x0) for message '' does not match selector known to Objective C runtime (0x7fca9b385200)-- abort |
So, in case there were any concerns with something with an I did some digging and it seems that absolutely everything had to be in place (duh!) for it to work properly. This included:
Oh joy.
|
I was able to repro this on master in the UIExplorer by enabling profiling and then clicking the reload button in the dev menu. I suspect it may have to do with the autogenerated profiling subclasses cc @tadeuzagallo |
Thanks @ide, that's the exact issue. I have a fix for it, should be out soon. |
Reopening this since the fix was reverted along with the diff before it: sahrens@f21e79d |
Facing the same issue on 0.8.0-rc2 and Xcode 7 beta 3. Any workaround to this temporarily? Finding it very difficult to do any sort of development without the ability to reload. |
Do you really need to reload during profile? You can put a |
Sorry, forgot to mention that I'm an Xcode newbie. Figured out that I had to do Build for -> Running rather than just hitting the play button (which was somehow building for profiling and causing the crash). Thanks! |
It's mostly useful for profiling how long the first render() takes, if you wanted to improve TTI for example. |
@ide, I understand and totally agree, it's just because he asked for a temporary workaround. I'm literally working on fixing it now, so it should be out soon. |
Updated my app to 0.6.0-rc and all hell breaks loose when I hit CMD + R. Also, SHIFT + CMD + Z (shake) does not invoke a menu any longer :(.
Thoughts?
The text was updated successfully, but these errors were encountered: