♻️ Simplify adding items to inventory by reusing 'AutoPlaceItemInInventory'

Source/inv.cpp

@@ -1335,9 +1335,7 @@ void CheckInvPaste(int pnum, int mx, int my)
 				SetCursor_(plr[pnum].HoldItem._iCurs + CURSOR_FIRSTITEM);
 			else
 				SetICursor(plr[pnum].HoldItem._iCurs + CURSOR_FIRSTITEM);
-			done2h = FALSE;
-			for (i = 0; i < NUM_INV_GRID_ELEM && !done2h; i++)
-				done2h = AutoPlaceItemInInventorySlot(pnum, i, plr[pnum].HoldItem, true);
+			done2h = AutoPlaceItemInInventory(pnum, plr[pnum].HoldItem, true);
 			plr[pnum].HoldItem = tempitem;
 			if (pnum == myplr)
 				SetCursor_(plr[pnum].HoldItem._iCurs + CURSOR_FIRSTITEM);

Source/items.cpp

@@ -1404,7 +1404,7 @@ void CreatePlrItems(int p)
 		{
 			SetPlrHandItem(&plr[p].HoldItem, IDI_WARRCLUB);
 			GetPlrHandSeed(&plr[p].HoldItem);
-			AutoPlaceItemInInventorySlot(p, 0, plr[p].HoldItem, TRUE);
+			AutoPlaceItemInInventory(p, plr[p].HoldItem, true);
 		}
 
 		SetPlrHandItem(&plr[p].SpdList[0], IDI_HEAL);

Source/stores.cpp

@@ -1311,10 +1311,7 @@ void S_SBuyEnter()
 				done = true;
 			}
 
-			for (i = 0; i < NUM_INV_GRID_ELEM && !done; i++) {
-				done = AutoPlaceItemInInventorySlot(myplr, i, plr[myplr].HoldItem, false);
-			}
-			if (done)
+			if (done || AutoPlaceItemInInventory(myplr, plr[myplr].HoldItem, false))
 				StartStore(STORE_CONFIRM);
 			else
 				StartStore(STORE_NOROOM);
@@ -1379,10 +1376,7 @@ void S_SPBuyEnter()
 				done = true;
 			}
 
-			for (i = 0; i < NUM_INV_GRID_ELEM && !done; i++) {
-				done = AutoPlaceItemInInventorySlot(myplr, i, plr[myplr].HoldItem, false);
-			}
-			if (done)
+			if (done || AutoPlaceItemInInventory(myplr, plr[myplr].HoldItem, false))
 				StartStore(STORE_CONFIRM);
 			else
 				StartStore(STORE_NOROOM);
@@ -1646,15 +1640,7 @@ void S_WBuyEnter()
 				done = TRUE;
 			}
 
-			for (i = 0; i < NUM_INV_GRID_ELEM && !done; i++) {
-				done = AutoPlaceItemInInventorySlot(myplr, i, plr[myplr].HoldItem, false);
-			}
-
-			if (!done) {
-				done = AutoPlaceItemInBelt(myplr, plr[myplr].HoldItem, false);
-			}
-
-			if (done)
+			if (done || AutoPlaceItemInInventory(myplr, plr[myplr].HoldItem, false) || AutoPlaceItemInBelt(myplr, plr[myplr].HoldItem, false))
 				StartStore(STORE_CONFIRM);
 			else
 				StartStore(STORE_NOROOM);
@@ -1829,8 +1815,8 @@ void S_BBuyEnter()
 		done = true;
 	}
 
-	for (int i = 0; i < NUM_INV_GRID_ELEM && !done; i++) {
-		done = AutoPlaceItemInInventorySlot(myplr, i, plr[myplr].HoldItem, false);
+	if (!done) {
+		done = AutoPlaceItemInInventory(myplr, plr[myplr].HoldItem, false);
 	}
 
 	StartStore(done ? STORE_CONFIRM : STORE_NOROOM);
@@ -1961,15 +1947,7 @@ void S_HBuyEnter()
 				done = TRUE;
 			}
 
-			for (i = 0; i < NUM_INV_GRID_ELEM && !done; i++) {
-				done = AutoPlaceItemInInventorySlot(myplr, i, plr[myplr].HoldItem, false);
-			}
-
-			if (!done) {
-				done = AutoPlaceItemInBelt(myplr, plr[myplr].HoldItem, false);
-			}
-
-			if (done)
+			if (done || AutoPlaceItemInInventory(myplr, plr[myplr].HoldItem, false) || AutoPlaceItemInBelt(myplr, plr[myplr].HoldItem, false))
 				StartStore(STORE_CONFIRM);
 			else
 				StartStore(STORE_NOROOM);