Review Request: Moved ClassDeclaration from KDevelop into KDevPlatform

Lior Mualem lior.m.kde at gmail.com
Sun Mar 15 14:19:50 UTC 2009


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

Review request for KDevelop.


Summary
-------

Moved ClassDeclaration from KDevelop into KDevPlatform as it's needed by many languages (and I need it for the class browser).
Also merged Cpp::DeclarationNavigationContext::htmlClass into AbstractDeclarationNavigationContext::htmlClass function - KDevPlatform has knowledge of the ClassDeclaration so there's no reason why it shouldn't be able to print out the class name and type (I actually fixed a todo in there :D ).

I guess there's more support code which can also be moved to KDevPlatform but I don't know enough in that area to do that. 
Maybe the language developers could gradually move the code around so it could be shared among everyone.


Diffs
-----

  /trunk/KDE/kdevelop/languages/cpp/codecompletion/item.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/codecompletion/missingincludeitem.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/codegen/cppnewclass.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/CMakeLists.txt 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/classdeclaration.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/classdeclaration.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/contextbuilder.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/contextbuilder.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/cppduchain.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/declarationbuilder.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/declarationbuilder.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/navigation/declarationnavigationcontext.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/navigation/declarationnavigationcontext.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/sourcemanipulation.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/templatedeclaration.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/tests/test_duchain.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/cppduchain/typeutils.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/tests/test_cppcodecompletion.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/stubconstructor.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/stubconstructor.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/stubcontextaction.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/stubcontextaction.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/tests/classdeclarationfactory.h 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/tests/classdeclarationfactory.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/tests/stubcreationtest.cpp 939540 
  /trunk/KDE/kdevelop/languages/cpp/veritas/uutcontextaction.cpp 939540 
  /trunk/KDE/kdevplatform/language/CMakeLists.txt 939543 
  /trunk/KDE/kdevplatform/language/duchain/classdeclaration.h 939543 
  /trunk/KDE/kdevplatform/language/duchain/classdeclaration.cpp 939543 
  /trunk/KDE/kdevplatform/language/duchain/navigation/abstractdeclarationnavigationcontext.cpp 939543 

Diff: http://reviewboard.kde.org/r/316/diff


Testing
-------


Thanks,

Lior





More information about the KDevelop-devel mailing list