Page MenuHomePhabricator

Exclusion certain methods from doxygen call graphs
Open, MediumPublicFeature

Description

Some methods, such as wfProfileIn(), wfProfileOut() don't really add much to a lot of the call graphs, and end up making them a lot more complex

Being able to exclude methods from being drawn would be really useful


Version: unspecified
Severity: enhancement

Details

Reference
bz26699

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:20 PM
bzimport set Reference to bz26699.
bzimport added a subscriber: Unknown Object (MLST).

Should probably be reported upstream. There are commands to enable the graphs on a per function basis \callgraph \callergraph . Maybe two new commands could be added to explicitly disable them such as \nocallgraph & \nocallergraph ?

Is this still a problem? maintenance/Doxyfile now specifies CALL_GRAPH = NO and CALLER_GRAPH = NO.

Reedy: Is this still a problem?

maintenance/Doxyfile now specifies CALL_GRAPH = NO and CALLER_GRAPH = NO.

Yes, well, sort of.

The call graphs have some use, but there are numerous methods that we don't really care if they are called - it gives no information.

They were turned off because they were too big etc

Krinkle set Security to None.
Krinkle removed a subscriber: Unknown Object (MLST).

Revisiting a decade old bug.

We had to:

maintenance/Doxyfile
CALL_GRAPH             = NO
CALLER_GRAPH           = NO

Looking at the Doxygen configuration https://www.doxygen.nl/manual/config.html , some methods can have their call or caller graphs generated using \callgraph or \callergraph.

And if we were to turn the options to YES again, methods can be opted out with \hidecallgraph and \hidecallergraph.

Those got added by https://github.com/doxygen/doxygen/pull/302 and released with Doxygen 1.8.10 (June 2015).


So essentially this task is fixed, upstream provided a way to hide some methods. I guess we can have a task to figure out whether we want to enable the call/caller graphs again and then hide some of the too generous ones. Else just mark this as resolved :)

Reedy renamed this task from Allow method blacklisting in doxygen call graphs to Allow exclusion of methods in doxygen call graphs.Apr 27 2021, 6:21 PM
Reedy updated the task description. (Show Details)
Reedy renamed this task from Allow exclusion of methods in doxygen call graphs to Exclusion certain methods from doxygen call graphs.Apr 27 2021, 6:24 PM
Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 12:24 PM