Review Request: Add mapping of Uses to AST nodes.

Bertjan Broeksema b.broeksema at home.nl
Thu Sep 24 15:38:57 UTC 2009


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

Review request for KDevelop, Andreas Pakulat, Milian Wolff, and David Nolden.


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
-----

  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/tests/test_duchain.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 

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