> On May 19, 2014, 3:57 p.m., David Edmundson wrote: > > freespacenotifier/freespacenotifier.cpp, line 66 > > <https://git.reviewboard.kde.org/r/118205/diff/1/?file=273471#file273471line66> > > > > set it to null too > > otherwise you can crash. > > > > Personally I'd also rename to m_ for everything, and consider using a > > QWeakPointer > > Martin Klapetek wrote: > Well it's a destructor...where would it crash? > > As for m_ - yes, I plan to, but I wanted this review to not contain > bazillion unrelated changes.
you're right. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118205/#review58157 ----------------------------------------------------------- On May 19, 2014, 4:15 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118205/ > ----------------------------------------------------------- > > (Updated May 19, 2014, 4:15 p.m.) > > > Review request for Plasma. > > > Repository: plasma-workspace > > > Description > ------- > > Currently it's done purely as a notification, which doesn't make much sense. > This patch turns it into a regular SNI. The actions were moved from the > notification popup to the SNI menu. > > > Diffs > ----- > > freespacenotifier/freespacenotifier.h d3c0351 > freespacenotifier/freespacenotifier.cpp cdf9bc5 > > Diff: https://git.reviewboard.kde.org/r/118205/diff/ > > > Testing > ------- > > Works properly. > > > File Attachments > ---------------- > > Screenshot (fonts are broken for some reason here) > > https://git.reviewboard.kde.org/media/uploaded/files/2014/05/19/a918c329-5679-4aeb-9b2e-af260c0bb4d6__fsn_sni.png > > > Thanks, > > Martin Klapetek > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
