Skip to content

Draft: Work/fix iterator (do not merge)

Ismael Asensio requested to merge work/fix_iterator into master

This branch/MR is about some analysis I did about the crashes !85 (merged) Is not meant to be merged as is. Just to get some more eyes on this.

  • First thing was to use remove() on the iterator instead of the hash. Anyway I suspect this path is not reached, but that on recycleWidget() instead.
  • Main cause for the crashes I found is that the iterator can turn around and start pushing items again, thus duplicating items in itemsToMove list, which makes the assert in line 1208 crash.
  • Lots of debug output to better follow the algorithm

Merge request reports