Review Request 118031: Reparse the project when includes/defines change (kdevplatform)

[Commit Hook]

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


This review has been submitted with commit fd0f3056154f401d2a6f9f3ea5b2872e609484f2 by Sergey Kalinichev to branch master.

- Commit Hook


On May 7, 2014, 10:03 a.m., Sergey Kalinichev wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118031/
> -----------------------------------------------------------
> 
> (Updated May 7, 2014, 10:03 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> Initially I wanted to use ParsingEnvironmentFile approach, but it seems like clang-plugin dosn't use all of it's functionallity (i.e it adds only the current file to the list of dependencies), also it's unclear to me how to handle defines in that approach. So instead I've decided to completly reparse the project.
> 
> 
> Diffs
> -----
> 
>   interfaces/iprojectcontroller.h 7ede76fc 
>   language/backgroundparser/parseprojectjob.h 8a6098b 
>   language/backgroundparser/parseprojectjob.cpp b0adc5e 
>   shell/projectcontroller.h 8c24376 
>   shell/projectcontroller.cpp 5257f49 
> 
> Diff: https://git.reviewboard.kde.org/r/118031/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Kalinichev
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140507/51bca53c/attachment.html>