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

Spurious spaces in container docs #21

Open
szhorvat opened this issue Jan 26, 2020 · 4 comments
Open

Spurious spaces in container docs #21

szhorvat opened this issue Jan 26, 2020 · 4 comments
Assignees

Comments

@szhorvat
Copy link
Member

Somehow we are ending up with spurious spaces in the container docs (e.g. vector_ copy instead of vector_copy).

image

@ntamas
Copy link
Member

ntamas commented Jan 26, 2020

Probably a bug in doxrox.py that surfaced after the code reformatting done by Astyle (Astyle inserted a space after commas in macros). I'll look into this.

@ntamas ntamas self-assigned this Jan 26, 2020
@ntamas
Copy link
Member

ntamas commented Jan 26, 2020

Fixed in igraph/igraph@39fc54e

@ntamas ntamas closed this as completed Jan 26, 2020
@ntamas
Copy link
Member

ntamas commented Jan 26, 2020

Hmmm, still not okay, now there's extra whitespace after the function name. I'll take care of this as well.

@ntamas ntamas reopened this Jan 26, 2020
@ntamas
Copy link
Member

ntamas commented Jan 26, 2020

Fixed the problem with the extra whitespace after the function name, but there are still some minor issues; basically, the function declaration in the documentation tries to preserve the formatting of the declaration from the original .pmt file - the problem is that we are replacing the macros from the .pmt file and this messes up the alignment of the function headers. We could try reformatting the .pmt files manually, but I don't think this is a priority now; we can do it any time, even after the release.

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