search

robotpy / robotpy-wpilib

Moved to https://github.com/robotpy/mostrobotpy
https://robotpy.github.io
Other
169 stars 59 forks source link

Integrate 2017 HAL changes #207

Closed virtuald closed 7 years ago

virtuald commented 7 years ago

There are a lot of changes to make here.

virtuald commented 7 years ago

Big change is that most HAL functions are prefixed by HAL_, so in 9bfcd10 I've changed the python HAL to convert foo to HAL_Foo, unless you explicitly specify which C function to import. The HAL scanner has been updated too.

virtuald commented 7 years ago

FYI, I've probably got about half of the HAL-level changes done -- there will be corresponding WPILib changes needed, but I haven't started on those yet.

virtuald commented 7 years ago

I've pushed my current progress to https://github.com/robotpy/robotpy-wpilib/tree/hal, heading to bed. The API signatures should match now.. but I haven't fixed some of the types or the actual implementation, or the WPILib changes needed to accomodate the hal changes.

virtuald commented 7 years ago

This is mostly fixed, the main thing left to do is to update the HAL scanner so that it can correctly detect enums and the handle type names, then I think the HAL scanner unit tests will pass, and this ticket can be closed.

virtuald commented 7 years ago

Fixed in 5abbbb2. The implementation is still not fixed, but that will just need to get done as the other 2017 updates are made.