spenap/butaca

segmentation fault when selecting a cinema in Showtimes View

mporsch opened this issue · 4 comments

mporsch commented

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
spenap commented

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

mporsch commented

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 commented

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 commented

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.