Review Request 119708: Try not to lose imports while reparsing.

[Milian Wolff]

> On Aug. 18, 2014, 5:52 p.m., Milian Wolff wrote:
> > I think this should go in (after the few nitpicks where resolved). I still hope we could maybe fix clang here instead of doing all this manually. But

...but a working kdev-clang is more important for now.


- Milian


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


On Aug. 16, 2014, 10:17 a.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119708/
> -----------------------------------------------------------
> 
> (Updated Aug. 16, 2014, 10:17 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-clang
> 
> 
> Description
> -------
> 
> Here is what happens:
> We are using CXTranslationUnit_PrecompiledPreamble flag, that makes clang internally use pch, but clang_getInclusions doesn't visit pch.
> 
> So let's try to extract this information manually
> 
> 
> Diffs
> -----
> 
>   duchain/clanghelpers.cpp 2c66421 
>   duchain/clangindex.cpp bd6c678 
>   tests/test_duchain.h 7844c17 
>   tests/test_duchain.cpp 563de7a 
> 
> Diff: https://git.reviewboard.kde.org/r/119708/diff/
> 
> 
> Testing
> -------
> 
> Added an unit test for it.
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140818/6cb40e6e/attachment.html>