> On Feb. 15, 2016, 10:06 a.m., Marco Martin wrote: > > src/plasmaquick/appletquickitem.cpp, line 55 > > <https://git.reviewboard.kde.org/r/127073/diff/2/?file=444222#file444222line55> > > > > why splitting it in a separate method if you don't call it from > > anywhere else?
Because d-pointer is accessed from `childEvent` which is called from AppletQuickItemPrivate constructor (`qmlObject` parented to `q`), and at that point `d` is dangling pointer. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127073/#review92387 ----------------------------------------------------------- On Feb. 13, 2016, 11:48 p.m., David Rosca wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127073/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2016, 11:48 p.m.) > > > Review request for Plasma. > > > Bugs: 358849 > http://bugs.kde.org/show_bug.cgi?id=358849 > > > Repository: plasma-framework > > > Description > ------- > > Regression from 344dbeb938 > > BUG: 358849 > > > Diffs > ----- > > src/plasmaquick/appletquickitem.h 1e0174a > src/plasmaquick/appletquickitem.cpp 28f1eb5 > src/plasmaquick/private/appletquickitem_p.h 1f99d2f > > Diff: https://git.reviewboard.kde.org/r/127073/diff/ > > > Testing > ------- > > Fixed for example plasmoid from bug. > Would be better if there was a way to fix it without adding another timer, > though. > > > Thanks, > > David Rosca > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
