Switch to C API - Githubissues

swri-robotics / marti_common

Common utility functions for MARTI

BSD 3-Clause "New" or "Revised" License

54 stars 63 forks source link

Switch to C API #689

Closed danthony06 closed 2 years ago

danthony06 commented 2 years ago

Resolves https://github.com/swri-robotics/marti_common/issues/688. @pjreed or @rjb0026 , would you mind taking a look at this? I added both the normal C API, and the re-entrant C API if we want to get potential speedups in the future.

pjreed commented 2 years ago

Sorry for not taking a look at this earlier, I've been away from computers for a while -- this looks pretty good to me, although I was just testing it and found a few things that need to be tweaked in order for this to be successfully imported into another package. I'll submit a PR with a few tweaks...