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

m.css crashes with AttributeError on Doxygen v1.9.1 #191

Closed
marcizhu opened this issue Feb 7, 2021 · 8 comments
Closed

m.css crashes with AttributeError on Doxygen v1.9.1 #191

marcizhu opened this issue Feb 7, 2021 · 8 comments

Comments

@marcizhu
Copy link

marcizhu commented Feb 7, 2021

For some reason, when using doxygen v1.9.X the script m.css/documentation/doxygen.py crashes with the following traceback:

Traceback (most recent call last):
  File "m.css/documentation/doxygen.py", line 3859, in <module>
    run(state, templates=os.path.abspath(args.templates), wildcard=args.wildcard, index_pages=args.index_pages, search_merge_subtrees=not args.search_no_subtree_merging, search_add_lookahead_barriers=not args.search_no_lookahead_barriers, search_merge_prefixes=not args.search_no_prefix_merging)
  File "m.css/documentation/doxygen.py", line 3716, in run
    parsed = parse_xml(state, file)
  File "m.css/documentation/doxygen.py", line 2988, in parse_xml
    func = parse_func(state, memberdef)
  File "m.css/documentation/doxygen.py", line 1965, in parse_func
    if signature.endswith('=default'):
AttributeError: 'NoneType' object has no attribute 'endswith'

As far as I know, this didn't happen with previous versions of Doxygen. I'm using m.css' latest commit (42d4a9a).

@crisluengo
Copy link
Contributor

This issue is fixed here:
#189

@marcizhu
Copy link
Author

marcizhu commented Feb 7, 2021

@crisluengo I have tested your PR and I can confirm that the changes proposed on #189 do indeed fix this issue.
@mosra Can we get that PR merged?

@marzer
Copy link
Contributor

marzer commented Apr 3, 2021

*bump*

@mosra I've just hit this bug too :(

@GiulioRomualdi
Copy link

I had the same problem

@marcizhu
Copy link
Author

@mosra Will this ever get fixed? It's been over four months already.

@mosra
Copy link
Owner

mosra commented Jan 9, 2022

Sorry sorry extremely sorry, I'm on this now. Finally was able to drag through the bug swamp that piled up over the past two years and now onto the final boss, Doxygen 1.9.

Or actually, it can wait one more day, right? Need some sleep.

@mosra
Copy link
Owner

mosra commented Jan 10, 2022

#189 is merged as c7ee89e, so things should no longer blow up. Until another Doxygen release, at least.

@mosra mosra closed this as completed Jan 10, 2022
@mosra
Copy link
Owner

mosra commented Jan 10, 2022

FYI for whoever is subscribed to this issue: #215 tracks remaining issues for proper Doxygen 1.9 support. Some of them are rather nasty (like the inline namespace regression), so until these boxes are all ticked off I don't recommend updating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

5 participants