alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Update esp_err.h

Browse Source 
Renamed the internal rc to __err_rc to avoid clashes with local variables.
This code would not do the expected thing with the original ESP_ERROR_CHECK macro:

esp_err_t my_func(esp_err_t x)
{
  assert(x == 23);
}

esp_err_t rc = 23; //some value that is important fo the user
ESP_ERROR_CHECK(my_func(rc));

The macro will expand to:
esp_err_t rc = (my_func(rc));

And the code will assert, as my_func will receive a random value - whatever is in the internal macro rc temp variable. This is due to the C weirdness of allowing this code:

int x = x; //x has a random value.
This commit is contained in:
jeanleflambeur 2017-10-12 09:53:09 +02:00 committed by Ivan Grokhotkov
parent a0cedb1f44
commit 8712fd3ccf
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
components/esp32/include/esp_err.h
View File

@ -64,14 +64,14 @@ void _esp_error_check_failed(esp_err_t rc, const char *file, int line, const cha
*/ */
#ifdef NDEBUG #ifdef NDEBUG
#define ESP_ERROR_CHECK(x) do { \ #define ESP_ERROR_CHECK(x) do { \
esp_err_t rc = (x); \ esp_err_t __err_rc = (x); \
(void) sizeof(rc); \ (void) sizeof(__err_rc); \
} while(0); } while(0);
#else #else
#define ESP_ERROR_CHECK(x) do { \ #define ESP_ERROR_CHECK(x) do { \
esp_err_t rc = (x); \ esp_err_t __err_rc = (x); \
if (rc != ESP_OK) { \ if (__err_rc != ESP_OK) { \
_esp_error_check_failed(rc, __FILE__, __LINE__, \ _esp_error_check_failed(__err_rc, __FILE__, __LINE__, \
__ASSERT_FUNC, #x); \ __ASSERT_FUNC, #x); \
} \ } \
} while(0); } while(0);