> On Dec. 1, 2015, 2:15 p.m., Martin Klapetek wrote: > >
Fixed, thanks; any other objections? - Sven ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126212/#review88999 ----------------------------------------------------------- On Dec. 1, 2015, 12:53 a.m., Sven Brauch wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126212/ > ----------------------------------------------------------- > > (Updated Dec. 1, 2015, 12:53 a.m.) > > > Review request for KDE Frameworks. > > > Bugs: 350785 > https://bugs.kde.org/show_bug.cgi?id=350785 > > > Repository: knotifications > > > Description > ------- > > See bug 350785. The crash is quite nasty, makes dbus allocate all available > file descriptors plus hangs the application; worth preventing imo. Yes, the > setup leading to this issue is broken, but it's just an env var and it's hard > to track down. > > > Diffs > ----- > > src/kstatusnotifieritem.cpp c48f4d0 > > Diff: https://git.reviewboard.kde.org/r/126212/diff/ > > > Testing > ------- > > Now prints an error message instead of crashing. > > > Thanks, > > Sven Brauch > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
