Skip to content

Imp/dot n plantuml improvements #11653

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

verheand
Copy link

@verheand verheand commented Jul 8, 2025

There are some improvements regarding speed especially for systems with expensive file i/o or calls to executables:

  • option DOT_DRY_RUN does not call dot from doxygen. Generation of files can be done afterwards e.g. using xargs dot.exe.
  • there's a look-up table that checks if the dot-graph has been already written to a file. This avoids file i/o checking if the file already exists.
  • the dot-files do not start with getOutputFileBase(). This reduces the number of files with identical content and also the time to render the graphs.
  • there's a look-up table that checks if the uml-graph has been already processed. If yes, it is skipped.
  • PlantUML file inline_umlgraph_* is now split up into multiple files. PlantUML supports only multi-threading for separated files. Thus, writing all graphs to one file uses only one thread in plantuml.
  • The inline_umlgraph files contain now a reference to the origin. This helps to fix syntax issues in the graphs.
  • Plantuml syntax errors are now reported as warnings.

This is my first pull request, so consider it as a draft proposal far away from been perfect and fully tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant