hashmismatch / freertos.rs

A Rust wrapper for FreeRTOS.
232 stars 22 forks source link

freertos_rs_delete_semaphore takes UBaseType_t in C wrapper #15

Closed uzytkownik closed 6 years ago

uzytkownik commented 6 years ago

A small type detected by compiling with warnings

// from shim.rs
extern {
    // ...
    pub fn freertos_rs_queue_delete(queue: FreeRtosQueueHandle);
    // ...

And in C code:

// queue.c
void vQueueDelete( QueueHandle_t xQueue )
// shim.c
void freertos_rs_queue_delete(UBaseType_t queue)

There are also a few minor problems:

I can send a pull request for them later today but they should be easy enough to fix

rudib commented 6 years ago

Should be fixed in the latest version of the shim, thanks for the report. I'll also probably modify the test environment to compile with -Wall to be really strict on such issues.