----------------------------------------------------------- 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 a.m., Marco Martin wrote: > > ----------------------------------------------------------- > 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, Plasma and Aleix Pol Gonzalez. > > > Repository: plasma-framework > > > Description > ------- > > This solves the crash > http://pastebin.com/tMYdgpUT > > the problem is that sometimes when QCoreApplication::aboutToQuit the > containment is already deleted (the order is not really deterministic), > therefore appletScript::applet() will hit an assert. > using a qpointer should solve it, its data being already 0ed in the cases the > containment is already deleted at :aboutToquit time. > > > Diffs > ----- > > src/scriptengines/qml/plasmoid/containmentinterface.h 3ecdddf > src/scriptengines/qml/plasmoid/containmentinterface.cpp f1762fe > > Diff: https://git.reviewboard.kde.org/r/119165/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
