New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong call graph in simple situation (Origin: bugzilla #721169) #5388

Closed
doxygen opened this Issue Jul 2, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@doxygen
Owner

doxygen commented Jul 2, 2018

status RESOLVED severity normal in component general for ---
Reported in version 1.8.6 on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2013-12-28 21:44:45 +0000, Kostya wrote:

Created attachment 264978
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

On 2014-01-08 13:34:08 +0000, Dimitri van Heesch wrote:

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.

On 2014-02-04 21:36:18 +0000, Kostya wrote:

It seems that the problem is still present

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

$ git log
commit SHA: 8f58d6d
Author: Dimitri van Heesch dimitri@stack.nl
Date: Tue Dec 24 19:54:25 2013 +0100

On 2014-02-04 21:57:17 +0000, Kostya wrote:

Ups. With latest version call graph seems to be OK! Thank you!

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

@doxygen doxygen closed this Jul 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment