Review Request 125554: Allow for classmodel users to define which data should be collected

[Aleix Pol Gonzalez]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125554/#review86502
-----------------------------------------------------------



language/classmodel/classmodel.h (line 60)
<https://git.reviewboard.kde.org/r/125554/#comment59583>

    Could use a better name for the type and flags. Maybe using something positive rather than No*?



language/classmodel/classmodel.h (line 125)
<https://git.reviewboard.kde.org/r/125554/#comment59582>

    Rename to m_features?


- Aleix Pol Gonzalez


On Oct. 8, 2015, 3:33 p.m., Sandro Andrade wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125554/
> -----------------------------------------------------------
> 
> (Updated Oct. 8, 2015, 3:33 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> The radial bundle view uses language/classmodel to show the
> project's namespaces, classes, and methods. For that purpose,
> it requires the ability to disable the following classmodel
> features:
> 
> 1) The gathering of information about derived and base classes.
> That would make radial bundle view enter an infinite loop.
> 
> 2) The gathering of information about classes which aren't part
> of the current opened project (the focus is only intra-project
> dependencies).
> 
> 3) The gathering of information about classes' internals (member
> functions and attributes). One of the radial bundle view's aims is to
> show dependencies between classes. Showing the methods for
> big projects would make the visualization cluttered, so that we
> need an option to not go into classes internals.
> 
> 
> Diffs
> -----
> 
>   language/classmodel/classmodel.h bb3331c 
>   language/classmodel/classmodel.cpp 535da1f 
>   language/classmodel/classmodelnode.cpp c9cd8d7 
> 
> Diff: https://git.reviewboard.kde.org/r/125554/diff/
> 
> 
> Testing
> -------
> 
> It builds fine, the behavior of class browser plugins isn't affected, and radial bundle is able to collected only the required information.
> 
> 
> Thanks,
> 
> Sandro Andrade
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20151008/04af518c/attachment.html>