ARMmbed / ble-nrf51822

Nordic stack and drivers for the mbed BLE_API
Other
46 stars 51 forks source link

line 81, explicit cast to (event_header_t *), caused error in ds-5 #13

Closed jslater8 closed 9 years ago

rgrover commented 9 years ago

Can you please discuss this change with me? As far as possible, we should avoid changing files inherited from the nordic SDK. Thanks.

jslater8 commented 9 years ago

generates an error in ds-5 when compiling, but it should work anyway without an explicit cast if the error is ignored.

rgrover commented 9 years ago

can you switch to more permissive compiler options when compiling the files under nordic-sdk? thanks.

rgrover commented 9 years ago

feel free to reopen if you feel this should be addressed properly.