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

Documentation fixes #5111

Merged
merged 2 commits into from
Jun 15, 2019
Merged

Documentation fixes #5111

merged 2 commits into from
Jun 15, 2019

Conversation

tiennou
Copy link
Contributor

@tiennou tiennou commented Jun 13, 2019

I've made some runs of my custom docurium, and here are the results.

I'm not sure how commit 1 is possible but you never know ^^. Note that I'm adding documentation for things which are just out of one-pass-declaration-parsing, so either we're fine with them as a first approximation, or I'll need help to clarify (a.k.a. help is welcome).

@tiennou
Copy link
Contributor Author

tiennou commented Jun 13, 2019

Not sure what to make of it, but there are a lots of holes caused by things I've "fixed" in docurium.

For example, our filter groups are different. I have hashsig, which doesn't show up at all now. I think that's the docparser-cleanups branch I have (passing the tempdir as the include path), I'll try to confirm. Not trying to be dense, I'm having a hard time remembering what's fixing what where how 😜.

On the other hand, git_odb_backend (whose "ours" version should look better after this PR gets in), which I ought to be still mangling because the typedef in "types.h" gets parsed after the struct "sys/odb_backend.h", overwriting the comments).

@pks-t
Copy link
Member

pks-t commented Jun 14, 2019

I'm not sure how commit 1 is possible but you never know ^^

You mean why there's no compile errors due to unknown prototypes/functions? Most users include "git2.h" directly which transitively includes all of our external headers, including "indexer.h". But docurium wouldn't do that, hence why it will probably fail to generate proper documentation here.

Copy link
Member

@pks-t pks-t left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks for all your nice documentation work! Not merging yet to give others a chance to chime in.

include/git2/repository.h Outdated Show resolved Hide resolved
include/git2/sys/filter.h Outdated Show resolved Hide resolved
Docurium seems to choke on this header because it can't see both
git_indexer_progress & git_indexer_progress_cb, let's add the include.
@tiennou
Copy link
Contributor Author

tiennou commented Jun 15, 2019

Rebased, should be ready to 🚢.

@ethomson
Copy link
Member

Looks good to me. Thanks, @tiennou!

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.

3 participants