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

Updated to Doxygen v1.8.9.1 and changed path 'dmd' to 'dmd2'. #846

Merged
merged 2 commits into from
Feb 20, 2015
Merged

Updated to Doxygen v1.8.9.1 and changed path 'dmd' to 'dmd2'. #846

merged 2 commits into from
Feb 20, 2015

Conversation

archfrog
Copy link
Contributor

The Doxyfile file hadn't been touched for 6 years so the paths referred to the old DMD project, effectively making the Doxyfile unusable. Not that it is worth a lot as it is, but even a little help to grok the project is valuable, IMHO.

@archfrog
Copy link
Contributor Author

  1. In-lined C++ source in Doxygen HTML docs for off-line study on a tablet.
  2. Increased DOT_GRAPH_MAX_NODES to 1,000 because Doxygen complained about it being too small. This may be ridiculously large, but it works well and doesn't take all that long to build the docs.
  3. Added driver and gen/passes subdirectories to make all sources available to Doxygen.

@archfrog archfrog closed this Feb 20, 2015
@archfrog archfrog reopened this Feb 20, 2015
dnadlinger added a commit that referenced this pull request Feb 20, 2015
Updated to Doxygen v1.8.9.1 and changed path 'dmd' to 'dmd2'.
@dnadlinger dnadlinger merged commit e4b2681 into ldc-developers:master Feb 20, 2015
@dnadlinger
Copy link
Member

Couldn't give it a thorough review yet, but it can hardly be worse than the previous state. Thanks!

@dnadlinger
Copy link
Member

Oh, and if you have any other ideas for documentation improvements, please keep the pull requests coming! The current state is sorely lacking in that regard.

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.

2 participants