From: Angus Gratton Date: Thu, 24 Nov 2016 04:09:19 +0000 (+0800) Subject: Merge branch 'bugfix/rmt_includes' into 'master' X-Git-Tag: v1.0~28 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3b937c5ef802fe6736876e539cfe189288538b6c;p=esp-idf 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 --- 3b937c5ef802fe6736876e539cfe189288538b6c