Re: Review Request 119165: store the containment as a qpointer

2014-07-08 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119165/ --- (Updated July 8, 2014, 9:05 a.m.) Status -- This change has been mar

Re: Review Request 119165: store the containment as a qpointer

2014-07-08 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119165/ --- (Updated July 8, 2014, 9:03 a.m.) Review request for KDE Frameworks, Plas

Re: Review Request 119165: store the containment as a qpointer

2014-07-08 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119165/#review61879 --- Ship it! Ship It! - David Edmundson On July 8, 2014, 8:36

Re: Review Request 119165: store the containment as a qpointer

2014-07-08 Thread David Edmundson
> On July 7, 2014, 10:37 p.m., David Edmundson wrote: > > src/scriptengines/qml/plasmoid/containmentinterface.cpp, line 99 > > > > > > If the containment is removed before the containmentInterface (the > > cause

Re: Review Request 119165: store the containment as a qpointer

2014-07-08 Thread Marco Martin
> On July 7, 2014, 10:37 p.m., David Edmundson wrote: > > src/scriptengines/qml/plasmoid/containmentinterface.cpp, line 99 > > > > > > If the containment is removed before the containmentInterface (the > > cause

Re: Review Request 119165: store the containment as a qpointer

2014-07-08 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119165/ --- (Updated July 8, 2014, 8:36 a.m.) Review request for KDE Frameworks, Plas

Re: Review Request 119165: store the containment as a qpointer

2014-07-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119165/#review61861 --- src/scriptengines/qml/plasmoid/containmentinterface.h