Review Request 119644: Delay setting top-context features for top context.

Sergey Kalinichev at
Thu Aug 7 11:31:55 UTC 2014

This is an automatically generated e-mail. To reply, visit:

(Updated Aug. 7, 2014, 3:31 p.m.)

Review request for KDevelop.


Guess it still makes sense to set those features for the context that's being updated.

Summary (updated)

Delay setting top-context features for top context.

Repository: kdev-clang


 setFeatures() for TopDUContext sets those features for ParsingEnvironmentFile too. And then when we check parsingEnvironmentFile()->featuresSatisfied(features) we always get true and as a result we get function bodies not parsed (see CXTranslationUnit_SkipFunctionBodies)

 The patch itself is very trivial, but I'm not familiar with the code at all, so please check whether it's the right way/place to fix it!

Diffs (updated)

  duchain/clanghelpers.cpp b1ee091 



All passed tests still pass.


Sergey Kalinichev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the KDevelop-devel mailing list