Review Request: Add mapping of Uses to AST nodes.
Bertjan Broeksema
b.broeksema at home.nl
Mon Sep 28 12:15:28 UTC 2009
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1710/
-----------------------------------------------------------
(Updated 2009-09-28 12:15:28.181649)
Review request for KDevelop, Andreas Pakulat, Milian Wolff, and David Nolden.
Changes
-------
Whoops I included some debug info that shouldn't be there in the previous patch.
Summary
-------
Adds a mapping of uses and AST nodes to ParseSession in addition to mappings between DUCHain declarations and nodes. I also factored out some code related to mappings out of DeclarationBuilder into ContextBuilder.
Diffs (updated)
-----
trunk/KDE/kdevelop/languages/cpp/cppduchain/contextbuilder.h 1027714
trunk/KDE/kdevelop/languages/cpp/cppduchain/contextbuilder.cpp 1027714
trunk/KDE/kdevelop/languages/cpp/cppduchain/declarationbuilder.h 1027714
trunk/KDE/kdevelop/languages/cpp/cppduchain/declarationbuilder.cpp 1027714
trunk/KDE/kdevelop/languages/cpp/cppduchain/usebuilder.cpp 1027714
trunk/KDE/kdevelop/languages/cpp/cppparsejob.cpp 1027714
trunk/KDE/kdevelop/languages/cpp/parser/parsesession.h 1027714
trunk/KDE/kdevelop/languages/cpp/parser/parsesession.cpp 1027714
trunk/KDE/kdevplatform/language/duchain/builders/abstractusebuilder.h 1027953
Diff: http://reviewboard.kde.org/r/1710/diff
Testing
-------
Work with a KDevelop that has this patch applied. No crashes related to this seem to happen. I did not yet test which nodes actually get mapped. Will do that soon.
Thanks,
Bertjan
More information about the KDevelop-devel
mailing list