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

Recycling through the message bases repeatedly breaks ddoc2.js #245

Open
dgets opened this issue Dec 8, 2015 · 2 comments
Open

Recycling through the message bases repeatedly breaks ddoc2.js #245

dgets opened this issue Dec 8, 2015 · 2 comments
Assignees
Labels

Comments

@dgets
Copy link
Owner

dgets commented Dec 8, 2015

While pounding the spacebar to make sure that the behavior I liked was sticking while working on issue #215, eventually I got an unhandled exception error due to the fact that msg_area.sub[user.cursub].name was undefined on line 747 of ddoc2.js. Going to wrap this in a try/catch loop and see if I can't find out some more details about it. Not sure if I should revert and find another bit to work on with #215 or if I should fix this and keep plugging away in this direction right now.

@dgets dgets added the bug label Dec 8, 2015
@dgets dgets self-assigned this Dec 8, 2015
@dgets dgets added this to the Version 0.2.x-beta milestone Dec 8, 2015
@dgets
Copy link
Owner Author

dgets commented Dec 9, 2015

ddoc-go-boom

Ugh. Throwing my own exception doesn't work here, nor does manually resetting user.cursub. I'm not sure what the hell to do on this other than that complete rewrite. As you can see scan pointers are blown and I believe that it's also reading out of different message bases that it shouldn't have access to. All of this just from hitting spacebar 10-15 times to cycle through the message bases.

@dgets
Copy link
Owner Author

dgets commented Feb 2, 2016

I'm starting to think that this might have to do by recursive calls opening mBase objects before they've closed the last instance of it opening. Might be a good idea to run through a check on all of the different opens and make sure that they're closed before passing to another method.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant