Not check the return value of esp_wifi_connect() in any case? Why?

[axellin]

In recent change of esp-idf, I found this commit:
https://github.com/espressif/esp-idf/co ... d1eac2f546

Can someone explain why?
Did this change fix any issue?

BTW,
I know what is changed by reading the code, but I don't know the *reason* for the change and what is the user visible impact.
Please improve the commit log to describe the *reason* for the changes.

[ESP_Sprite]

From our internal commit: 'esp_wifi_connect() is called after triggering the WiFi disconnect event when the WiFi is connected, and esp_wifi_connect() returns ERR. In this case, we should not check the return value of esp_wifi_connect()'

My guess would be that this works around some issues that would make the examples way more complicated if handled correctly.