No that's all.  It doesn't show up in the active list now.  Thanks.<br><br><div class="gmail_quote">On Tue, Jul 7, 2009 at 11:03 PM, Robert Knight <span dir="ltr"><<a href="mailto:robertknight@gmail.com">robertknight@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
> On 2009-07-07 07:00:39, Kurt Hindenburg wrote:<br>
> > If this patch has been applied, please close this on reviewboard.<br>
<br>
This patch has been marked as submitted - do I need to do anything else?<br>
<br>
<br>
- Robert<br>
<br>
<br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
<a href="http://reviewboard.kde.org/r/802/#review1481" target="_blank">http://reviewboard.kde.org/r/802/#review1481</a><br>
-----------------------------------------------------------<br>
<br>
<br>
On 2009-06-17 16:22:24, mick22 wrote:<br>
><br>
> -----------------------------------------------------------<br>
> This is an automatically generated e-mail. To reply, visit:<br>
> <a href="http://reviewboard.kde.org/r/802/" target="_blank">http://reviewboard.kde.org/r/802/</a><br>
> -----------------------------------------------------------<br>
><br>
> (Updated 2009-06-17 16:22:24)<br>
><br>
><br>
> Review request for Konsole.<br>
><br>
><br>
> Summary<br>
> -------<br>
><br>
> Here's a patch that drastically reduces mem usage when a fixed size history is<br>
> used:<br>
> - new history storage type "HistoryTypeQList"  (QList-based) set as replacement for HistoryTypeBuffer<br>
> - compact storage for history lines<br>
><br>
> History lines are stored as an array of chars when possible (i.e. when<br>
> color and rendition flag of all characters in the line are equal). 2 bytes per character + overhead<br>
><br>
> When not (easily) possible, it falls back to the current way of storing history lines (array of QVector<Character>). 11 bytes per character + overhead<br>
><br>
> Memory usage has been cut down from about 171MB to 55MB for a konsole process<br>
> with 5 tabs, 30000 lines scrollback, 65 chars per line.<br>
><br>
> Update:<br>
> As suggested in <a href="https://bugs.kde.org/show_bug.cgi?id=176974#c20" target="_blank">https://bugs.kde.org/show_bug.cgi?id=176974#c20</a>, a formatting array has been introduced in v2 of the patch. The CompactHistoryLine class can now handle single- and multi-color history lines. This replaces v1 of the patch where 2 separate classes were used.<br>

><br>
><br>
> This addresses bug 176974.<br>
>     <a href="https://bugs.kde.org/show_bug.cgi?id=176974" target="_blank">https://bugs.kde.org/show_bug.cgi?id=176974</a><br>
><br>
><br>
> Diffs<br>
> -----<br>
><br>
>   /trunk/KDE/kdebase/apps/konsole/src/Character.h 981222<br>
>   /trunk/KDE/kdebase/apps/konsole/src/History.h 981222<br>
>   /trunk/KDE/kdebase/apps/konsole/src/History.cpp 981222<br>
>   /trunk/KDE/kdebase/apps/konsole/src/SessionController.cpp 981222<br>
>   /trunk/KDE/kdebase/apps/konsole/src/SessionManager.cpp 981222<br>
><br>
> Diff: <a href="http://reviewboard.kde.org/r/802/diff" target="_blank">http://reviewboard.kde.org/r/802/diff</a><br>
><br>
><br>
> Testing<br>
> -------<br>
><br>
> reasonably tested<br>
><br>
><br>
> Thanks,<br>
><br>
> mick22<br>
><br>
><br>
<br>
</blockquote></div><br>