Skip to content
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

Add doxygen documentation to header files #10

Open
peanutfun opened this issue Nov 10, 2021 · 1 comment
Open

Add doxygen documentation to header files #10

peanutfun opened this issue Nov 10, 2021 · 1 comment

Comments

@peanutfun
Copy link

Description

Doxygen is a common and very useful tool for building a documentation of C++ source code based on reading code comments in a specific format. It helps keeping the code documentation "with the code", meaning that one is less likely to update the code, but not its documentation.

GiNaCDE provides a type of "public API" by exposing certain variables and functions through its header files. Most of them are currently listed in the documentation.pdf under "Some useful commands in library". I suggest documenting at least these variables and commands with Doxygen as well.

CMake has a module which searches for Doxygen and provides useful functions for building the documentation.

Proposal

  • Add doxygen docstrings to variables and commands in the GiNaCDE headers that are considered "public API", i.e. interacted with by users.
  • Make CMake build the Doxygen documentation (if Doxygen is found on the host system)

Related issues

openjournals/joss-reviews#3885

@mithun218
Copy link
Owner

@peanutfun thanks for your suggestion. I am sorry to say that currently, I am entirely ignorant of Doxygen. From your explanation, it looks excellent to use Doxygen in GiNaCDE. I hope that I will be able to use Doxygen to GiNaCDE very soon.

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

No branches or pull requests

2 participants