Review Request 119642: Make opened files be parsed properly

Milian Wolff mail at milianw.de
Thu Aug 7 12:30:18 UTC 2014


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


Ah sorry, now that I think more about this, the forced update is _wrong_

You probably saw this when using kdev-clang, right? The bug is there, it does not check the old environment vs. the new one and then thinks no update is required. Always forcing an update of all opened files will be slow and we do not want to do that.

- Milian Wolff


On Aug. 7, 2014, 10:33 a.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119642/
> -----------------------------------------------------------
> 
> (Updated Aug. 7, 2014, 10:33 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> First queue active document with highest priority, then currently opened files and everything else. That way we get active document parsed first of all.
> Also forcefully reparse all opened files to make sure they have valid include directories (otherwise sometimes those files won't be parsed properly)
> 
> 
> Diffs
> -----
> 
>   language/backgroundparser/parseprojectjob.cpp 7f430d7 
>   shell/projectcontroller.cpp 994a737 
> 
> Diff: https://git.reviewboard.kde.org/r/119642/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140807/57485e77/attachment.html>


More information about the KDevelop-devel mailing list