----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105940/#review17369 -----------------------------------------------------------
Ship it! Makes sense. Thanks! - Sebastian Kügler On Aug. 9, 2012, 1:34 a.m., Simeon Bird wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105940/ > ----------------------------------------------------------- > > (Updated Aug. 9, 2012, 1:34 a.m.) > > > Review request for Plasma. > > > Description > ------- > > As noted in bug 256187, the fileWatcher plasmoid can use a lot of memory, > particularly when first reading a file. > This turns out to be because the whole file is added to the textDocument, > instead of just the last few lines. > > As a bonus, fix an (unreported) bug where if a line matched two filters, it > could be printed twice. > > > This addresses bug 256187. > http://bugs.kde.org/show_bug.cgi?id=256187 > > > Diffs > ----- > > applets/fileWatcher/fileWatcher.cpp fdf804c > > Diff: http://git.reviewboard.kde.org/r/105940/diff/ > > > Testing > ------- > > Compiled, watched some (small and large) files, added and deleted lines from > them. > > > Thanks, > > Simeon Bird > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
