Review Request: Class browser re-design

Lior Mualem lior.m.kde at gmail.com
Sat Mar 21 10:05:31 UTC 2009


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

Review request for KDevelop.


Summary
-------

Almost a complete re-write of the class model.
I did it mostly because I wanted to move the logic away from the model and into the node class. 
This will eventually allow for greater flexibility with different node types and different operations for each node type - Two examples would be the BaseClassesFolderNode and the DerivedClassesFolderNode, both which implements an informative view for the node that they belong to.

Also implemented is a dynamic population of the nodes in the tree - information is retrieved only when needed and when the node is collapsed, it's sub-items are cleared (only for dynamic nodes which is most of the nodes).

Some stuff still left to be done before it's considered usable (in my opinion anyway):
 * Updates - I don't perform any update at the moment, once a project is loaded, it stays that way (one way around it is to collapse the node and re-expand it. It will force a refresh).
 * Search capabilities - I plan on having a special "Search results" node which will show the results for the current search.


Diffs
-----

  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/CMakeLists.txt 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classbrowserplugin.h 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classbrowserplugin.cpp 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classmodel.h 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classmodel.cpp 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classmodelnode.h 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classmodelnode.cpp 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classmodelnodescontroller.h PRE-CREATION 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classmodelnodescontroller.cpp PRE-CREATION 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classtree.h 939887 
  /trunk/playground/devtools/kdevelop4-extra-plugins/classbrowser/classtree.cpp 939887 

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


Testing
-------


Thanks,

Lior





More information about the KDevelop-devel mailing list