Review Request 118265: Cache the top-level context associated with a QML module name

[Commit Hook]

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


This review has been submitted with commit a79fdead61d8250846f357ecf8c84b9ea2277a79 by Denis Steckelmacher to branch master.

- Commit Hook


On May 22, 2014, 7:28 p.m., Denis Steckelmacher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118265/
> -----------------------------------------------------------
> 
> (Updated May 22, 2014, 7:28 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-qmljs
> 
> 
> Description
> -------
> 
> As pointed out by Sven Brauch, KStandardDirs::findResource can be slow (it has to access the file system). This patch caches in the parse session the top-level context of every referenced module. This way, when the user types and the same parse session is reused over and over again to update the DUChain, the filesystem is not touched.
> 
> 
> Diffs
> -----
> 
>   duchain/parsesession.h 0222d72 
>   duchain/parsesession.cpp 798de10 
> 
> Diff: https://git.reviewboard.kde.org/r/118265/diff/
> 
> 
> Testing
> -------
> 
> All the unit tests still pass, and the behavior of import statements is as before (maybe a bit quicker, but it may be an impression).
> 
> 
> Thanks,
> 
> Denis Steckelmacher
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140523/2b49e13f/attachment.html>