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

Doxygen: Support inlining simple structs in file #83

Open
burningmime opened this issue Jan 15, 2019 · 3 comments
Open

Doxygen: Support inlining simple structs in file #83

burningmime opened this issue Jan 15, 2019 · 3 comments
Labels
3rd party Requires a modification to a 3rd party project. help wanted Suitable for a community contribution. laterz Will get resolved eventually, but not now.

Comments

@burningmime
Copy link

Doxygen supports the INLINE_SIMPLE_STRUCTS configuration which is great for plain C APIs where a struct is just 2 fields together. It seems m.css does not honor this (maybe Doxygen is the one that isn't generating the XML correctly).

If this isn't feasible for you to implement, is there a simple way I could pre-process the XML to get this?

@mosra mosra added this to TODO in Doxygen theme via automation Feb 2, 2019
@mosra
Copy link
Owner

mosra commented Feb 2, 2019

Hi, sorry for the delayed reply, I am rather busy this year :)

I actually attempted to implement support for this, but as far as I remember (from a year ago), the generated XML showed no difference if this option was enabled or not and so I put it aside. I think this behavior didn't change at all since 1.8.13 but frankly I'm not sure.

The thing is is -- this option affects how internal links (and also tag files) are generated, so while preprocessing the XML could work, it would probably run into a lot of nasty corner cases (invalid links etc), making it all very ugly. I'm interested in using this option for my projects as well but the support has to be done on Doxygen side first -- could you check if this option is reflected in the generated XML and if not (which is my assumption), open a feature request on the Doxygen project itself?

After the Doxygen side is done, I can add support here as well.

Thank you!

@burningmime
Copy link
Author

No problem; thanks for getting back. I checked, and it still has no effect on the output. It does not inline them in the output.

Doxygen theme automation moved this from TODO to Done Feb 7, 2019
@mosra mosra reopened this Feb 7, 2019
Doxygen theme automation moved this from Done to In Progress Feb 7, 2019
@mosra mosra added the help wanted Suitable for a community contribution. label Feb 7, 2019
@mosra
Copy link
Owner

mosra commented Feb 7, 2019

I see you opened an issue on the Doxygen repo: doxygen/doxygen#6825 Reopening so I (or someone else) can get back to this once the Doxygen part is implemented.

@mosra mosra added the 3rd party Requires a modification to a 3rd party project. label Feb 7, 2019
@mosra mosra moved this from In Progress to 3rd party TODO in Doxygen theme Feb 7, 2019
@mosra mosra added the laterz Will get resolved eventually, but not now. label Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3rd party Requires a modification to a 3rd party project. help wanted Suitable for a community contribution. laterz Will get resolved eventually, but not now.
Projects
Doxygen theme
  
3rd party TODO
Development

No branches or pull requests

2 participants