Review Request 124003: Speed up grep output view

Aleix Pol Gonzalez aleixpol at kde.org
Thu Jun 4 10:46:41 UTC 2015



> On June 4, 2015, 12:11 p.m., 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?
> 
> Oleksandr Senkovych wrote:
>     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.
> 
> Aleix Pol Gonzalez wrote:
>     So the difference is the scrollbar, right?
> 
> Oleksandr Senkovych wrote:
>     Yeah, it is.

Well, if it could make sense to do it only one after the job is done.
All these resizeColumnToContents calls are there so that the output stays visible as a whole, obviously.


- Aleix


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


On June 4, 2015, 11:56 a.m., Oleksandr Senkovych wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124003/
> -----------------------------------------------------------
> 
> (Updated June 4, 2015, 11:56 a.m.)
> 
> 
> 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/32776a60/attachment-0001.html>


More information about the KDevelop-devel mailing list