----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122184/#review74518 -----------------------------------------------------------
Ship it! Looks good to me, although it sounds like something Qt should be taking care of.... - David Faure On Jan. 21, 2015, 4:57 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122184/ > ----------------------------------------------------------- > > (Updated Jan. 21, 2015, 4:57 p.m.) > > > Review request for Plasma, David Faure and Hugo Pereira Da Costa. > > > Repository: oxygen > > > Description > ------- > > According to the comment we need to delete the qstyle object before the > plugin if the application exits with exit(). A small test showed this is > no longer the case. > > More importantly this patch fixes a crash we see when styles were switched at > runtime and the app uses exit(). > > Because Breeze has a similar delete line the first would run deleting > the style object but leaving a dangling pointer, the second would run > and crash. > > Even if they only deleted their own style, both need to check the active > style which means the second one is always accessing a dangling pointer. > > ---- > > A similar approach needs to go into Breeze too. I shall copy that if this > gets in. > > > Diffs > ----- > > kstyle/oxygenstyleplugin.cpp 409ec13 > > Diff: https://git.reviewboard.kde.org/r/122184/diff/ > > > Testing > ------- > > Wrote a unit test that calls exit(0) on button press. This used to crash if > we changed theme and then pressed the button. Now it no longer crashes ever. > > Checked that kglobalaccel5 no longer crashes on exit after changing > Breeze->Oxygen and does not crash after changing from Oxygen->Breeze > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel