Review Request 119642: Make opened files be parsed properly

[Milian Wolff]

> On Aug. 7, 2014, 12:30 p.m., Milian Wolff wrote:
> > 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.

note: the rest of this patch is OK, i.e. preferring to first update the active document, then the opened ones, then the others.


- Milian


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


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/99783060/attachment-0001.html>