diff --git a/dCommon/GeneralUtils.h b/dCommon/GeneralUtils.h index f59ec71f..35c0b895 100644 --- a/dCommon/GeneralUtils.h +++ b/dCommon/GeneralUtils.h @@ -158,9 +158,7 @@ namespace GeneralUtils { template [[nodiscard]] std::optional TryParse(std::string_view str) { numeric_parse_t result; - if (!str.empty()) { - while (std::isspace(str.front())) str.remove_prefix(1); - } + while (!str.empty() && std::isspace(str.front())) str.remove_prefix(1); const char* const strEnd = str.data() + str.size(); const auto [parseEnd, ec] = std::from_chars(str.data(), strEnd, result); @@ -186,9 +184,7 @@ namespace GeneralUtils { template [[nodiscard]] std::optional TryParse(std::string_view str) noexcept try { - if (!str.empty()) { - while (std::isspace(str.front())) str.remove_prefix(1); - } + while (!str.empty() && std::isspace(str.front())) str.remove_prefix(1); size_t parseNum; const T result = details::_parse(str, parseNum); diff --git a/dGame/dGameMessages/GameMessages.cpp b/dGame/dGameMessages/GameMessages.cpp index 3ef9f3b5..74946179 100644 --- a/dGame/dGameMessages/GameMessages.cpp +++ b/dGame/dGameMessages/GameMessages.cpp @@ -4660,7 +4660,7 @@ void GameMessages::HandleBuyFromVendor(RakNet::BitStream& inStream, Entity* enti if (!user) return; Entity* player = Game::entityManager->GetEntity(user->GetLoggedInChar()); if (!player) return; - + // handle buying normal items auto* vendorComponent = entity->GetComponent(); if (vendorComponent) { @@ -5290,7 +5290,7 @@ void GameMessages::HandleRemoveItemFromInventory(RakNet::BitStream& inStream, En bool iLootTypeSourceIsDefault = false; LWOOBJID iLootTypeSource = LWOOBJID_EMPTY; bool iObjIDIsDefault = false; - LWOOBJID iObjID; + LWOOBJID iObjID = LWOOBJID_EMPTY; bool iObjTemplateIsDefault = false; LOT iObjTemplate = LOT_NULL; bool iRequestingObjIDIsDefault = false;