After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 721169 - Wrong call graph in simple situation
Wrong call graph in simple situation
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.6
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2013-12-28 21:44 UTC by Kostya
Modified: 2014-02-04 21:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
code with doxygen config (83.12 KB, application/zip)
2013-12-28 21:44 UTC, Kostya
Details

Description Kostya 2013-12-28 21:44:45 UTC
Created attachment 264978 [details]
code with doxygen config

I has two similar functions PrintGnuPlotIndex and PrintGnuPlotSpectra doing very similar things. No templates, pointers, ifdefs and so on - just construct and object and use it inside function. But the call graph is incorrect. They bouth use gnuplot::GnuplotWrapper::MakeOutput(), however PrintGnuPlotSpectra has it in call graph, and PrintGnuPlotIndex does not.

I had stiped down my fully functional project to the minimal and very simple example - the bug is present. I had also tested official Linux binary 1.8.6 and 1.8.0 with the same result. Just copy binary to doc/doxygen/ and ./run-doxygen.sh to generate html.

My code and call graph example is attached
Comment 1 Dimitri van Heesch 2014-01-08 13:34:08 UTC
Confirmed. 

The problem was the C++11 {...} style initializer for addMultiPoint, i.e.
wrapper.AddMultiPoint({i+1.0, best_x[i]});

Should be fixed in the next GIT update.
Comment 2 Kostya 2014-02-04 21:36:18 UTC
It seems that the problem is still present

make distclean
git pull
./configure --english-only
make

$ git log
commit 8f58d6dd7c3d0f6004d127bf111f76e4a4478516
Author: Dimitri van Heesch <dimitri@stack.nl>
Date:   Tue Dec 24 19:54:25 2013 +0100
Comment 3 Kostya 2014-02-04 21:57:17 UTC
Ups. With latest version call graph seems to be OK! Thank you!


commit 5b68436fbcc1d45eef63783faae8727e9d806096
Merge: 89638fb 3df3176
Author: Dimitri van Heesch <dimitri@stack.nl>
Date:   Tue Feb 4 21:55:23 2014 +0100