> On nov. 17, 2014, 9:45 p.m., Thomas Lübking wrote: > > attica-kde/kdeplugin/kdeplatformdependent.cpp, line 56 > > <https://git.reviewboard.kde.org/r/121161/diff/2/?file=328928#file328928line56> > > > > is > > > > KdePlatformDependent::~KdePlatformDependent() > > { > > if (QCoreApplication::closingDown()) > > m_accessManager->setParent(nullptr); > > } > > > > sufficient?
Isn't that a more complex way of doing the same? - Albert ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121161/#review70542 ----------------------------------------------------------- On nov. 17, 2014, 9:11 p.m., Jeremy Whiting wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121161/ > ----------------------------------------------------------- > > (Updated nov. 17, 2014, 9:11 p.m.) > > > Review request for kdelibs and Plasma. > > > Repository: plasma-desktop > > > Description > ------- > > Plugin unload order was making the attica_kde plugin crash on close, > this works around it by leaking one AccessManager. > > > Diffs > ----- > > attica-kde/kdeplugin/kdeplatformdependent.cpp > 489c03b18b7bb940007ab51cb197105fbc25de9f > > Diff: https://git.reviewboard.kde.org/r/121161/diff/ > > > Testing > ------- > > knewstuff tests no longer crash on close. > > > Thanks, > > Jeremy Whiting > >