[Kde-pim] Review Request: Workaround crash on exit with Qt 4.8

[Oswald Buddenhagen]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102981/#review7742
-----------------------------------------------------------



akregator/src/folder.cpp
<http://git.reviewboard.kde.org/r/102981/#comment6642>

    you don't need the temporary; just delete children.takeFirst();
    
    this is no workaround, but a proper fix. TreeNode's d'tor modifies children, which invalidates qDeleteAll's iterator.
    
    an alternative fix is saving the child list to a temporary and clearing the list, then deleteAll on the temporary. this may be faster (possible removeAll()s in the d'tor go over an empty list), but has some potential to break things if the d'tor relies on actually seeing all children.


- Oswald Buddenhagen


On Oct. 27, 2011, 11:47 p.m., Christoph Feck wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102981/
> -----------------------------------------------------------
> 
> (Updated Oct. 27, 2011, 11:47 p.m.)
> 
> 
> Review request for KDEPIM, Christophe Giboudeaux and Frank Osterfeld.
> 
> 
> Description
> -------
> 
> Workaround crash on exit with Qt 4.8. Didn't see a bug report for it yet, but there is a related one for plasma at https://bugs.kde.org/show_bug.cgi?id=284989
> 
> 
> Diffs
> -----
> 
>   akregator/src/folder.cpp f2385b0 
> 
> Diff: http://git.reviewboard.kde.org/r/102981/diff/diff
> 
> 
> Testing
> -------
> 
> Works.
> 
> 
> Thanks,
> 
> Christoph Feck
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/