Skip to content

Commit

Permalink
Remove some debug notes
Browse files Browse the repository at this point in the history
  • Loading branch information
lbussy committed Jul 29, 2020
1 parent 8d0bd37 commit cf6696f
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
3 changes: 1 addition & 2 deletions src/jsonconfig.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,6 @@ bool deserializeConfig(Stream &src)
config.load(doc.as<JsonObject>());
return true;
}
// TODO: Can I return false here somehow?
}

bool serializeConfig(Print &dst)
Expand All @@ -117,7 +116,7 @@ bool serializeConfig(Print &dst)
JsonObject root = doc.to<JsonObject>();

// Fill the object
config.save(root); // TODO: I think this is why it's not working right, it's reloading the config
config.save(root);

// Serialize JSON to file
return serializeJsonPretty(doc, dst) > 0;
Expand Down
1 change: 0 additions & 1 deletion src/thatVersion.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ bool deserializeVersion(const char * &src, ThatVersion &thatVersion) {
thatVersion.load(doc.as<JsonObject>());
return true;
}
// TODO: Can/should I return false here somehow?
}

void doPoll() {
Expand Down
3 changes: 0 additions & 3 deletions src/webserver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -423,9 +423,6 @@ void setSettingsAliases()

// Parse JSON

// TODO: Use NULL checks from jsonconfig
// TODO: Can I use jsonconfig to handle this?

// Parse Access Point Settings Object
const char *ssid = doc["apconfig"]["ssid"];
if ((ssid) && (strcmp(ssid, config.apconfig.ssid) != 0))
Expand Down

0 comments on commit cf6696f

Please sign in to comment.