<div dir="auto"><div>Hi,</div><div dir="auto"><br></div><div dir="auto">as Christoph said, these patches are very much appreciated. Please don't hesitate to make larger changes that improve the Plugin. Indeed, the code is old and should be improved.</div><div dir="auto"><br></div><div dir="auto">Best regards</div><div dir="auto">Dominik<br><br><div class="gmail_quote" dir="auto"><div dir="ltr">Dr.-Ing. Christoph Cullmann <<a href="mailto:cullmann@absint.com">cullmann@absint.com</a>> schrieb am So., 22. Juli 2018, 16:10:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I appreciate the work on the plugin.<br>
<br>
It would make sense if you submit your patches in phabricator.<br>
<br>
<a href="https://phabricator.kde.org/" rel="noreferrer noreferrer" target="_blank">https://phabricator.kde.org/</a><br>
<br>
That way, one can review them in a more nice way and we don't spam other<br>
list members ;=)<br>
<br>
Greetings<br>
Christoph<br>
<br>
----- Am 22. Jul 2018 um 15:01 schrieb loh.tar <a href="mailto:loh.tar@googlemail.com" target="_blank" rel="noreferrer">loh.tar@googlemail.com</a>:<br>
<br>
> Hello!<br>
> <br>
> These small series is a byproduct while I have written a new parser type which<br>
> you will find in some other thread.<br>
> <br>
> Most of these patches do not fix a really big bug but do things a little bit<br>
> better. So, at the end of the day there are a lot CPU cycles saved and maybe<br>
> some nerves of the user spared.<br>
> <br>
> BTW: The related code looks a little abandoned but I resist e.g the temptation<br>
> to run these 'astyle' command, noted somewhere in your coding guidelines, or to<br>
> make bigger alterations. There are many possibilities for improvements.<br>
> <br>
> Because this is my very first post to some mailing list like yours, please<br>
> forgive if I have done something not as I should.<br>
> <br>
> Thanks, Lothar<br>
> <br>
> loh.tar (8):<br>
>  SymbolView: Fix broken toggle actions<br>
>  SymbolView: Reduce redundant code<br>
>  SymbolView: Qt docu recommends to populate view with disabled sorting<br>
>  SymbolView: Avoid unneeded update of current item<br>
>  SymbolView: Update current item and sorting after parsing<br>
>  SymbolView: Avoid unneeded update of current item II<br>
>  SymbolView: Reduce flicker of view in tree mode when edit text<br>
>  SymbolView: Fix: Find new current item when symbol is in first line<br>
> <br>
> .../symbolviewer/plugin_katesymbolviewer.cpp  | 101 ++++++++++--------<br>
> addons/symbolviewer/plugin_katesymbolviewer.h |   3 +<br>
> 2 files changed, 61 insertions(+), 43 deletions(-)<br>
> <br>
> --<br>
> 2.18.0<br>
<br>
-- <br>
----------------------------- Dr.-Ing. Christoph Cullmann ---------<br>
AbsInt Angewandte Informatik GmbH      Email: cullmann@AbsInt.com<br>
Science Park 1                         Tel:   +49-681-38360-22<br>
66123 Saarbrücken                      Fax:   +49-681-38360-20<br>
GERMANY                                WWW:   <a href="http://www.AbsInt.com" rel="noreferrer noreferrer" target="_blank">http://www.AbsInt.com</a><br>
--------------------------------------------------------------------<br>
Geschäftsführung: Dr.-Ing. Christian Ferdinand<br>
Eingetragen im Handelsregister des Amtsgerichts Saarbrücken, HRB 11234<br>
</blockquote></div></div></div>