fix: reduce function body

opentibiabr · Nov 19, 2024 · e04c208 · e04c208
1 parent 2e34768
commit e04c208
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/src/server/network/protocol/protocolgame.cpp b/src/server/network/protocol/protocolgame.cpp
@@ -5746,18 +5746,13 @@ void ProtocolGame::sendForgeHistory(uint8_t page) {
 	auto historyVector = player->getForgeHistory();
 	auto historyVectorLen = historyVector.size();
 
-	uint16_t currentPage = 1;
-	uint16_t lastPage = 1;
-	uint16_t pageFirstEntry = 0;
-	uint16_t pageLastEntry = 0;
-
 	std::vector<ForgeHistory> historyPerPage;
 	if (historyVectorLen > 0) {
-		lastPage = std::clamp<uint16_t>(std::floor((historyVectorLen - 1) / 9) + 1, 0, std::numeric_limits<uint16_t>::max());
-		currentPage = (lastPage < page) ? lastPage : page;
+		uint16_t lastPage = std::clamp<uint16_t>(std::floor((historyVectorLen - 1) / 9) + 1, 0, std::numeric_limits<uint16_t>::max());
+		uint16_t currentPage = (lastPage < page) ? lastPage : page;
 
-		pageFirstEntry = std::clamp<uint16_t>(historyVectorLen - (currentPage - 1) * 9, 0, std::numeric_limits<uint16_t>::max());
-		pageLastEntry = historyVectorLen > currentPage * 9 ? std::clamp<uint16_t>(historyVectorLen - currentPage * 9, 0, std::numeric_limits<uint16_t>::max()) : 0;
+		uint16_t pageFirstEntry = std::clamp<uint16_t>(historyVectorLen - (currentPage - 1) * 9, 0, std::numeric_limits<uint16_t>::max());
+		uint16_t pageLastEntry = historyVectorLen > currentPage * 9 ? std::clamp<uint16_t>(historyVectorLen - currentPage * 9, 0, std::numeric_limits<uint16_t>::max()) : 0;
 
 		for (uint16_t entry = pageFirstEntry; entry > pageLastEntry; --entry) {
 			historyPerPage.emplace_back(historyVector[entry - 1]);