Results 348 comments of Dustin Spicuzza
trafficstars

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...