Little improvements

steam_user_stats: Use the value stored in the JSON object.
steal_inventory: use 'empty' that is guaranteed to be constant in time.
This commit is contained in:
Nemirtingas 2019-10-20 13:46:25 +02:00
parent 42d9217f00
commit ef5693d9f8
2 changed files with 5 additions and 11 deletions

View file

@ -840,7 +840,7 @@ bool SubmitUpdateProperties( SteamInventoryUpdateHandle_t handle, SteamInventory
void RunCallbacks() void RunCallbacks()
{ {
if (call_definition_update || inventory_requests.size()) { if (call_definition_update || !inventory_requests.empty()) {
if (!item_definitions_loaded) { if (!item_definitions_loaded) {
read_items_db(); read_items_db();
item_definitions_loaded = true; item_definitions_loaded = true;
@ -854,7 +854,7 @@ void RunCallbacks()
call_definition_update = false; call_definition_update = false;
} }
if (inventory_requests.size() && !inventory_loaded) { if (!inventory_requests.empty() && !inventory_loaded) {
read_inventory_db(); read_inventory_db();
inventory_loaded = true; inventory_loaded = true;
} }

View file

@ -350,9 +350,7 @@ const char * GetAchievementDisplayAttribute( const char *pchName, const char *pc
return static_cast<std::string const&>(item["name"]) == pchName; return static_cast<std::string const&>(item["name"]) == pchName;
}); });
if (it != defined_achievements.end()) { if (it != defined_achievements.end()) {
static std::string display_name; return it.value()["displayName"].get_ptr<std::string*>()->c_str();
display_name = it.value()["displayName"].get<std::string>();
return display_name.c_str();
} }
} catch (...) {} } catch (...) {}
} }
@ -363,9 +361,7 @@ const char * GetAchievementDisplayAttribute( const char *pchName, const char *pc
return static_cast<std::string const&>(item["name"]) == pchName; return static_cast<std::string const&>(item["name"]) == pchName;
}); });
if (it != defined_achievements.end()) { if (it != defined_achievements.end()) {
static std::string description; return it.value()["description"].get_ptr<std::string*>()->c_str();
description = it.value()["description"].get<std::string>();
return description.c_str();
} }
} catch (...) {} } catch (...) {}
} }
@ -376,9 +372,7 @@ const char * GetAchievementDisplayAttribute( const char *pchName, const char *pc
return static_cast<std::string const&>(item["name"]) == pchName; return static_cast<std::string const&>(item["name"]) == pchName;
}); });
if (it != defined_achievements.end()) { if (it != defined_achievements.end()) {
static std::string hidden; return it.value()["hidden"].get_ptr<std::string*>()->c_str();
hidden = it.value()["hidden"].get<std::string>();
return hidden.c_str();
} }
} catch (...) {} } catch (...) {}
} }