]> granicus.if.org Git - esp-idf/commit
Fixes for stdlib.h inclusion
authorAngus Gratton <angus@espressif.com>
Wed, 23 Nov 2016 21:08:09 +0000 (08:08 +1100)
committerAngus Gratton <angus@espressif.com>
Wed, 23 Nov 2016 21:20:21 +0000 (08:20 +1100)
commit79646f41b596f56cc26aa1c3df9697c71867df4a
treef267b231e7f0430a81675fc2a853a03988c478bd
parent137c0272741e0166898f07b104edf8dcce80446d
Fixes for stdlib.h inclusion

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 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 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.
components/driver/rmt.c
components/freertos/include/freertos/FreeRTOSConfig.h
components/mbedtls/port/esp_bignum.c
examples/04_https_request/main/https_request_main.c