-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Crashes on destruction #242
Comments
Thanks for reporting, I'll have a look. |
I have a backtrace from the Spectacle crash, in case that helps: |
Thanks, I'll have a look into this issue tomorrow. |
Downstream bug report: https://bugs.kde.org/show_bug.cgi?id=437674 |
I have removed deletes of widgets and layouts where a parent was provided on construction. Would be nice if someone could test. |
That does seem to have fixed it, Spectacle no longer crashes when closing. |
Awesome, version is tagged now. |
I also confirm the spectacle segfault is gone, thanks! |
There is too much deleting of widgets and layouts going on, in particular the issue seems to be in FontPicker but there's too many deletes everywhere. In Qt you don't need to delete widgets or layouts to which you have given a parent, the destructor will take care of deleting them, you can delete them manually but you have to be careful of not double deleting, i.e if you delete mLayout, that will already delete mButtonLayout because it is it's child
mLayout->addLayout(mButtonLayout)
so when you do laterdelete mButtonLayout
again it crashes because you have already deleted it.My suggestion would be to not do manual deleting of QWidgets and just left the parent/child relationship cleanup everything.
The text was updated successfully, but these errors were encountered: