Review Request: reserve size

David Nolden david.nolden.kde at art-master.de
Sat Feb 26 11:57:28 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100737/#review1660
-----------------------------------------------------------

Ship it!


Looks good, feel free to push this. If you don't have a developer account, tell us.

- David


On Feb. 24, 2011, 10:58 p.m., Floris Ruijter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100737/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2011, 10:58 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> by calling reserve upfront some time is saved. 
> 
> 
> Diffs
> -----
> 
>   languages/cpp/parser/rpp/chartools.cpp 91612bd 
> 
> Diff: http://git.reviewboard.kde.org/r/100737/diff
> 
> 
> Testing
> -------
> 
> i did several tests with duchainify on the cpp subtree to find the fastest denominator (it's 10)
> 
> no prealloc
> real	4m47.104s
> user	1m31.478s
> sys	0m16.549s
> 
> prealloc 1/8
> real	4m35.194s
> user	1m30.786s
> sys	0m16.341s
> 
> prealloc 1/9
> real	4m34.568s
> user	1m30.870s
> sys	0m15.917s
> 
> prealloc 1/10
> real	4m32.682s
> user	1m29.150s
> sys	0m15.941s
> 
> prealloc 1/11
> real	4m35.614s
> user	1m28.390s
> sys	0m15.949s
> 
> prealloc 1/12
> real	4m39.231s
> user	1m27.613s
> sys	0m16.209s
> 
> 
> Thanks,
> 
> Floris
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110226/06ae21f1/attachment.html>


More information about the KDevelop-devel mailing list