> On May 21, 2014, 7:36 a.m., Sebastian Kügler wrote: > > freespacenotifier/freespacenotifier.cpp, line 105 > > <https://git.reviewboard.kde.org/r/118205/diff/3/?file=273777#file273777line105> > > > > I wonder if fsInfo.size() can ever be 0? Please check that, in order to > > avoid divide by zero.
It should not, otherwise the check above (fsInfo.isValid()) should return false. > On May 21, 2014, 7:36 a.m., Sebastian Kügler wrote: > > freespacenotifier/freespacenotifier.cpp, line 138 > > <https://git.reviewboard.kde.org/r/118205/diff/3/?file=273777#file273777line138> > > > > This could be compile-time as well. I plan to change the style in this file altogether afterwards, so I'll include that. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118205/#review58212 ----------------------------------------------------------- On May 20, 2014, 4:17 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118205/ > ----------------------------------------------------------- > > (Updated May 20, 2014, 4:17 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
