Alternative to ESP_ERROR_CHECK

[kolban]

The ESP-IDF provides a useful macro called ESP_ERROR_CHECK() that takes a statement as a parameter. If the statement does NOT return ESP_OK, then we assert and log the line. Unfortunately, this doesn't include the error code value that was actually returned by the statement.

Here is an alternative definition which logs the error code in addition to throwing an assertion:

Code: Select all

#undef ESP_ERROR_CHECK
#define ESP_ERROR_CHECK(x)   do { esp_err_t rc = (x); if (rc != ESP_OK) { ESP_LOGE("err", "esp_err_t = %d", rc); assert(0 && #x);} } while(0);

[ESP_Angus]

Hi Kolban,

Good addition. Could you please send this as a pull request on github? We'll likely incorporate it into esp-idf.

Angus

[BuddyCasino]

Very nice! This has bugged me too, didn't know it is that easy to contribute.

[Bmillier]

Hi Neil
This is off topic: I love your book. Have used it on several projects, the latest being a learning IR remote/ wifi blaster using RMT peripheral. I note on page 255 you show max values of 65535 for the 15- bit timing value saved by the RMT peripheral. This should be 32767.
Using the Arduino IDE and my own drivers, I have gotten the IR transmit function working fine using
LIRC conf files for the IR data structure, as a test. I just got the basic IR receive function working for the "learning" part, but there's more work needed to handle the wide range of IR signals that the different remotes emit.
This would be easier to do if I were using the Espressif IDF, but that's more than I want to bite off right now.

[kolban]

Thank you sir. Documentation changes to 32767.

[jeanleflambeur]

Pls use a mangled name for the internal macro variable rc.
That's because in C/C++ this statement actually compiles:

Code: Select all

int a = a;

It results in variable a being uninitialized essentially.
So with your macro, this code would have unexpected results:

Code: Select all

int rc = 7;
ESP_ERROR_CHECK(initialize_remote_control(rc));

The rc value passed to initialize_remote_control is actually random, not 7.

Also, this code would not compile because what is passed to the function is the internal esp_err_t instead of the string:

Code: Select all

esp_err_t  initialize_db(const string& name)
{
	//...
}
int  main(int argc, const char* argv[])
{
	string rc = "db1";
	ESP_ERROR_CHECK(initialize_db(rc));
	return 0;
}

I have a PR about this issue without any answer so far: https://github.com/espressif/esp-idf/pull/1109

The solution is simple - just call that internal variable smth else like __internal_rc__