8000 Update ESP8266WiFiSTAClass::waitForConnectResult() for the same behavior with ESP32. by ShinDaiIchi · Pull Request #4985 · esp8266/Arduino · GitHub
[go: up one dir, main page]
Skip to content

Update ESP8266WiFiSTAClass::waitForConnectResult() for the same behavior with ESP32. #4985

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Update ESP8266WiFiSTAClass::waitForConnectResult() for the same behavior with ESP32. #4985

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
0be5be6
Update ESP8266WiFiSTAClass::waitForConnectResult() for the same beha…
ShinDaiIchi Jul 30, 2018
d2c4cc7
Add td_split() for spliting String to String Array by multi-delimi…
ShinDaiIchi Jul 30, 2018
b39fc18
Add td_split() for spliting String to String Array
ShinDaiIchi Jul 30, 2018
579a9a6
Update WString.h
ShinDaiIchi Jul 30, 2018
5c4ff27
Update WString.cpp
ShinDaiIchi Jul 30, 2018
8abc913
Merge branch 'master' into master
devyte Aug 1, 2018
8544d30
Update ESP8266WiFiSTAClass::waitForConnectResult with connection's …
ShinDaiIchi Aug 2, 2018
956510e
Update ESP8266WiFiSTAClass::waitForConnectResult with connection's …
ShinDaiIchi Aug 2, 2018
52fcfa3
Merge branch 'master' into master
devyte Aug 9, 2018
c5fec54
Update (wait_secs==0)? etc...
ShinDaiIchi Aug 10, 2018
cba45ea
Merge branch 'master' into master
d-a-v Sep 19, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update (wait_secs==0)? etc...
  • Loading branch information
@ShinDaiIchi
ShinDaiIchi authored Aug 10, 2018
commit c5fec545e3755f4a7a5a6f5b219eafa46380c5a3
5 changes: 3 additions & 2 deletions libraries/ESP8266WiFi/src/ESP8266WiFiSTA.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -374,8 +374,9 @@ uint8_t ESP8266WiFiSTAClass::waitForConnectResult(uint32_t wait_secs) {
if((wifi_get_opmode() & 1) == 0) {
return WL_DISCONNECTED;
}
int i = 0; int try_times = wait_secs*10;
while((!status() || status() >= WL_DISCONNECTED) && ((wait_secs!=0)? (i++ < try_times): true )) {
int i = 0;
int try_times = wait_secs*10;
while((!status() || status() >= WL_DISCONNECTED) && ((wait_secs==0)? true : (i++ < try_times))) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is difficult to read.

  • !status() tests whether the status is non-zero, but status is an enum and it’s unobvious which value is zero.
  • The part after && is really complicated.

How about:

for (int timeout = wait_secs * 10; wait_secs == 0 || timeout > 0; --timeout) {
  auto st = status();
  switch (st) {
    case WL_CONNECTED:
    case WL_NO_SSID_AVAIL:
    case WL_CONNECT_FAILED:
      return st;
    default:
      delay(100);
      break;
  }
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree with @yoursunny . However:

I have a PR with a class that encapsulates the polled timeout. I suggest waiting a bit until it's merged, then using it, instead of reinventing the wheel for the nth time.
the switch should define cases for all enum values, and leave the default case for values not in the enum (error).

delay(100);
}
return status();
Expand Down
0