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