2008/10/21 Aaron J. Seigo <[EMAIL PROTECTED]>: > On Tuesday 21 October 2008, Guillaume Pothier wrote: >> Hi, >> DesktopView::setContainment is sometimes explicitly called with a null >> containment, > > hm. when does that happen? when changing the Activity via the conig dialog? or > switching between existing Activities?
I don't remember exactly the sequences of actions that trigger the crash, I just stumbled upon that fix while tidying up my plasma/kephal patches. But it was probably something related to adding/removing monitors. You can see setContainment(0) statements in DesktopView::screenOwnerChanged and PlasmaApp::adjustSize. >> however this is not checked in the code. The attached >> patch solves that. Ok to commit? > > yep... Oka > > -- > Aaron J. Seigo > humru othro a kohnu se > GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43 > > KDE core developer sponsored by Qt Software > > > > _______________________________________________ > Plasma-devel mailing list > Plasma-devel@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
