softdevice: fix writing to a characteristic

[aykevl]

This seems to have been broken since #192. I suspect it's a problem with the struct calling convention (which most certainly is a TinyGo CGo bug, but at least this works around the problem).

This fixes my Write() call on the PineTime that previously returned an error.

[HattoriHanzo031]

This seems to have been broken since #192. I suspect it's a problem with the struct calling convention (which most certainly is a TinyGo CGo bug, but at least this works around the problem).

This fixes my Write() call on the PineTime that previously returned an error.

Interesting, I presume sd_ble_gatts_hvx_noescape does not have this problem because it creates the struct inside the function like your workaround? sd_ble_gap_adv_start_noescape is also implemented similar to sd_ble_gatts_value_set_noescape. I didn't see any issue with it, but it is only for NRF51 which might be the reason it in not affected. How exactly does the issue manifest, maybe I can try to test if it also affects sd_ble_gatts_value_set_noescape.

[deadprogram]

Thanks for the fix @aykevl now merging.

[aykevl]

How exactly does the issue manifest, maybe I can try to test if it also affects sd_ble_gatts_value_set_noescape.

It was returning an error and not actually doing what it is supposed to do.

This is a chance this worked by accident on nrf51 just not on the nrf52.