]> granicus.if.org Git - esp-idf/commit
Merge branch 'bugfix/rmt_includes' into 'master'
authorAngus Gratton <angus@espressif.com>
Thu, 24 Nov 2016 04:09:19 +0000 (12:09 +0800)
committerAngus Gratton <angus@espressif.com>
Thu, 24 Nov 2016 04:09:19 +0000 (12:09 +0800)
commit3b937c5ef802fe6736876e539cfe189288538b6c
treef267b231e7f0430a81675fc2a853a03988c478bd
parent137c0272741e0166898f07b104edf8dcce80446d
parent79646f41b596f56cc26aa1c3df9697c71867df4a
Merge branch 'bugfix/rmt_includes' into 'master'

Fix inclusion of stdlib.h

Refs:
http://esp32.com/viewtopic.php?f=13&t=550
http://esp32.com/viewtopic.php?f=13&t=551

rmt.c should include stdlib.h for malloc, esp_bignum,c & https_request_main.c should include it for abort().

FreeRTOSConfig.h is only including stdlib if CONFIG_FREERTOS_ASSERT_ON_UNTESTED_FUNCTION is set. However, it is included for abort() so needs to be included whenever CONFIG_FREERTOS_ASSERT_FAIL_ABORT is set.

This change includes it unconditionally in FreeRTOSConfig.h. This is to avoid this kind of bug where compiler errors are dependent on config. I suggest we don't change this to be more selective until we have 'make randomconfig' style tests in CI.

See merge request !244