]> granicus.if.org Git - esp-idf/commitdiff
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)
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


Trivial merge