espressif / esp-nimble

A fork of NimBLE stack, for use with ESP32 and ESP-IDF
Apache License 2.0
76 stars 50 forks source link

Fix recursive call / stack overflow #15

Closed corecode closed 3 years ago

corecode commented 3 years ago

fix recursive call, which can not be tail call optimized on esp32

CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.

corecode commented 3 years ago

This fix probably should be upstreamed as well.

corecode commented 3 years ago

This was merged upstream as https://github.com/apache/mynewt-nimble/pull/857

dhrishi commented 3 years ago

Hi @corecode Sorry for the late reply. Now that this is merged upstream, we will pull this PR from there and get it included in esp-nimble. As a result, you can close this PR.

dhrishi commented 3 years ago

@corecode And, Thanks for your contribution! :-)

dhrishi commented 3 years ago

Updated esp-nimble to include this support from upstream: https://github.com/espressif/esp-nimble/commit/b703caaefd39cd91f02e52ac75d0845596ac1094

Closing the PR!