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

Kevin Funk kfunk at kde.org
Fri May 9 17:54:36 UTC 2014



> On May 8, 2014, 5:29 a.m., Pär-Ola Nilsson wrote:
> > s/m_foreUpdate/m_forceUpdate/g
> >

Fixed


- Kevin


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


On May 7, 2014, 2:16 p.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, 2:16 p.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/20140509/272c04be/attachment.html>


More information about the KDevelop-devel mailing list