Review Request 124003: Speed up grep output view

[Oleksandr Senkovych]

> On Червень 4, 2015, 10:11 до полудня, Aleix Pol Gonzalez wrote:
> > Looks interesting!
> > Do you know what's exactly the part that is making your views struggle? Why isn't resizeColumnToContents needed anymore?

The reason it's so laggy is exactly in calling resizeColumnToContents on every insert and it's geting worse with each new item. Specifically, there is a raising number of item delegate sizeHint method calls.

Actually I did just a little bit more tesing and it seemes that there is a difference - if line does not fit into the kdevelop window if you don't resize a column as visible on theese screenshot:
http://i.imgur.com/1Cgmdxn.png (to the left - with patch applied, to the right - without).

Actually maybe there is a way to still keep previous behaviour - call resizeColumnToContents once after the grep job is finished.


- Oleksandr


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124003/#review81168
-----------------------------------------------------------


On Червень 4, 2015, 9:56 до полудня, Oleksandr Senkovych wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124003/
> -----------------------------------------------------------
> 
> (Updated Червень 4, 2015, 9:56 до полудня)
> 
> 
> Review request for KDevelop and Kevin Funk.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> Grep output view is very laggy if there is a lot of items. For example on my machine adding just 300 items to this view takes about 10 seconds and whole KDevelop UI becomes unresponsive. After applying proposed changes there is no problem to add even 10 000 items - everything works without any lag. 
> Also, I didn't notice any regression in the UI: all rows in the view are displayed properly.
> 
> 
> Diffs
> -----
> 
>   plugins/grepview/grepoutputview.h 5191c58 
>   plugins/grepview/grepoutputview.cpp 2fba779 
> 
> Diff: https://git.reviewboard.kde.org/r/124003/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Oleksandr Senkovych
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150604/5f255cf3/attachment-0001.html>