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

segmentation fault when selecting a cinema in Showtimes View #20

Open
mporsch opened this issue Apr 10, 2014 · 4 comments
Open

segmentation fault when selecting a cinema in Showtimes View #20

mporsch opened this issue Apr 10, 2014 · 4 comments
Labels

Comments

@mporsch
Copy link
Collaborator

mporsch commented Apr 10, 2014

steps to reproduce:

  1. open Showtimes View
  2. select one cinema
  3. dwelve deep into the Movie Browse or Movie Views
  4. return to Showtimes View
  5. select a different cinema
@mporsch mporsch added the bug label Apr 10, 2014
@spenap
Copy link
Owner

spenap commented Apr 10, 2014

This is the same one as issue #17 , so I would start investigating around how I created the QAbstractListModels.

@mporsch
Copy link
Collaborator Author

mporsch commented Apr 13, 2014

delete m_currentMovieListModel;
in
https://github.com/spenap/butaca/blob/master/src/theaterlistmodel.cpp#L114
seems to delete a resource that is still in use in the QML components. At least when commenting it out there are no more crashes.

@spenap
Copy link
Owner

spenap commented Apr 23, 2014

Yes, I think I got to know that back then. That could be a good enough fix: a memory leak coming from there shouldn't get too big, as typically users would close the app every once in a while. And that's much better than an unexpected crash, so we can comment that delete out, and leave a comment pointing here.

@mporsch
Copy link
Collaborator Author

mporsch commented May 3, 2014

I commmented out the corresponding line in commit
1fa26f8

I leave this issue open until a better solution is implemented, e.g. the QML components pull a copy of their own to work with.

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

2 participants