Dustin Spicuzza
Dustin Spicuzza
it's going to be really difficult to fix this, better to migrate to cxxheaderparser instead; but I want to finish commands v2 before starting on that.
Definitely a bug. I don't really have an interest in fixing this myself (lots of stuff going on), but I can provide guidance for debugging/fixing this if you would like...
I find that setting the environment variable `CPPHEADERPARSER_DEBUG=1` is really useful for figuring out what the parser is doing.
Thanks for the contribution! It's likely that I won't have time to review until Saturday evening, but I'll do my best.
I definitely forgot about this! I'm traveling, but will add it to my calendar for when I get home. There's nothing obviously wrong as far as I can see, but...
... definitely forgot again. Adding it back to my calendar, I don't have anything urgent planned tonight so I should be able to do it then.
This PR failed to parse one of the core RobotPy libraries that we use CppHeaderParser for. I'll dig in to see if I can isolate it.
[cxxheaderparser](https://github.com/robotpy/cxxheaderparser) does many of the same things as this (and is better at handing some C++11+ features), but intentionally doesn't do some of the weird ctypes/type resolving things that CppHeaderParser...
Definitely a bug. I don't really have an interest in fixing this myself (lots of stuff going on), but I can provide guidance for debugging/fixing this if you would like...
There's currently a `classes` dict and a `classes_order` list, I would expect it to contain the parsed classes even if `classes` doesn't contain it. Could add a `classes_ns` dictionary that...