----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120327/#review67270 -----------------------------------------------------------
wallpapers/image/backgroundlistmodel.cpp <https://git.reviewboard.kde.org/r/120327/#comment46961> hmm, this way you are saying if the package b is not valid, then try to access its metadata. doesn't look correct.. - Marco Martin On Sept. 22, 2014, 11:38 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120327/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2014, 11:38 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > Fix crash in ImageWallpaper > > When not in a package (i.e a lone image) the metadata file cannot be > loaded. This returns an invalid metadata (KPluginInfo) with isValid() > set to false but an invalid d pointer. > > > Diffs > ----- > > wallpapers/image/backgroundlistmodel.cpp a667c10 > > Diff: https://git.reviewboard.kde.org/r/120327/diff/ > > > Testing > ------- > > In config to change the wallpaper click the remove button of one of your own > images, it didn't crash. > > It fixes it, but I'm not convinced it's the correct fix, we're still spewing > warnings about the package being broken. > > > Thanks, > > David Edmundson > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
