-
Notifications
You must be signed in to change notification settings - Fork 183
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
A fatal exception occurred #133
Comments
What version were you using, and/or which commit hash did you build from? |
I failed to take note at the moment but considering I do a Current debugger build is 0.9.8 alpha which is most likely the one when it crashed. Suggestion: I really like the idea of message box popping up with the callstack at the time of the crash, can we also include a build version/number in there so the issue reporter don't have to remember? |
Hi, I'm pretty sure I encountered the same bug but on the latest commit. I did however have a look at it in remedy and it said that "commit_row" was null on the same line for the latest commit.
|
This looks like it's being caused by starting an edit on the value, expression, or view rule of a row in a watch or watch-like tab, scrolling away from it, then clicking somewhere else (thus ending & attempting to commit an edit, for a row which is no longer in view). There are two options for the debugger here: (a) it can go through with the commit anyways (I'd have to move away from using an actually visualized row's information, since the row which is being edited is out-of-bounds and thus has no UI built for it), or (b) I can abort the edit if the row is out of bounds. I've gone ahead with (b). While it's somewhat idiosyncratic with the way edits work while a row is in view (e.g. where clicking away actually commits the edit), it's a bit more conservative w.r.t. making non-obvious changes to the user, and so I'm leaning in that direction. So, long story short, this is fixed in e637f72. |
I have had this bug occur again recently
|
@achalpandeyy You are on a version that was released before this fix was pushed. |
I was going back and forth the "Memory" and "Watch" windows when the exception occurred.
Below is the message copied from the message box.
The text was updated successfully, but these errors were encountered: