ElixirForge/WiFiManager
1
0
Fork 0
mirror of https://github.com/alexhopeoconnor/WiFiManager.git synced 2026-10-13 14:51:21 +10:00
Code Releases Activity

Use hasParam() instead of checking server.arg() for NULL, it returns a String anyway.

Browse Source 
Don't use .c_str() when its being assigned to a String anyway
This commit is contained in:
Christopher Liebman
2018-07-15 10:44:06 -07:00
parent fc84e7a260
commit 85dd747949
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
WiFiManager.cpp
+6 -2
View File
@@ -1228,9 +1228,13 @@ void WiFiManager::doParamSave(){
break;
}
//read parameter from server
String name = (String)FPSTR(S_parampre)+(String)i;
String value;
if(server->arg((String)FPSTR(S_parampre)+(String)i) != NULL) value = server->arg((String)FPSTR(S_parampre)+(String)i).c_str();
else value = server->arg(_params[i]->getID()).c_str();
if(server->hasArg(name)) {
value = server->arg(name);
} else {
value = server->arg(_params[i]->getID());
}
//store it in params array
value.toCharArray(_params[i]->_value, _params[i]->_length+1); // length+1 null terminated