Review Request: C++ implementation of the Checking data types. See the other patch about the KDevPlatform changes.

Aleix Pol Gonzalez aleixpol at gmail.com
Tue Nov 15 01:35:08 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103140/
-----------------------------------------------------------

Review request for KDevelop.


Description
-------

This patch shows the additions and changes in the KDevelop C++ support to provide the data structures used for increased verbosity on the code.

See: https://git.reviewboard.kde.org/r/103071/

In this patch I guess the most controversial part is the ParseSession::typeFromCallAst addition. I think it's useful to have, but probably we can sort it out in some other way by querying the DUChain. I'd prefer to know what do you guys think.

Thanks!

PS: In this patch I'd appreciate if you concentrate on the actual changes being done instead of the style. ;)


Diffs
-----

  languages/cpp/cppduchain/CMakeLists.txt 5f27b8f 
  languages/cpp/cppduchain/controlflowgraphbuilder.h PRE-CREATION 
  languages/cpp/cppduchain/controlflowgraphbuilder.cpp PRE-CREATION 
  languages/cpp/cppduchain/expressionvisitor.h d62495d 
  languages/cpp/cppduchain/expressionvisitor.cpp 9379285 
  languages/cpp/cppduchain/name_visitor.cpp d15769e 
  languages/cpp/cppduchain/tests/CMakeLists.txt 363404f 
  languages/cpp/cppduchain/tests/codeanalysistest.h PRE-CREATION 
  languages/cpp/cppduchain/tests/codeanalysistest.cpp PRE-CREATION 
  languages/cpp/cppduchain/tests/test_duchain.h d4b0edf 
  languages/cpp/cppduchain/tests/testhelper.h 61a3de2 
  languages/cpp/cppduchain/tests/testhelper.cpp fd58d4b 
  languages/cpp/cppduchain/usebuilder.cpp f3f5926 
  languages/cpp/cppduchain/usedecoratorvisitor.h PRE-CREATION 
  languages/cpp/cppduchain/usedecoratorvisitor.cpp PRE-CREATION 
  languages/cpp/cppparsejob.h df748cd 
  languages/cpp/cppparsejob.cpp 2401f6e 
  languages/cpp/parser/parsesession.h 2859a43 
  languages/cpp/parser/parsesession.cpp d3669ff 

Diff: http://git.reviewboard.kde.org/r/103140/diff/diff


Testing
-------

See the unit tests.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111115/6668a482/attachment.html>


More information about the KDevelop-devel mailing list