- made CallTryPickup a global function.

This one still gets called in too many places but stands in the way of moving Inventory fully to the script side.

src/g_inventory/a_pickups.cpp

@@ -260,11 +260,11 @@ DEFINE_ACTION_FUNCTION(AInventory, DoRespawn)
 //
 //===========================================================================
 
-bool AInventory::CallTryPickup(AActor *toucher, AActor **toucher_return)
+bool CallTryPickup(AInventory *item, AActor *toucher, AActor **toucher_return)
 {
 	static VMFunction *func = nullptr;
 	if (func == nullptr) PClass::FindFunction(&func, NAME_Inventory, NAME_CallTryPickup);
-	VMValue params[2] = { (DObject*)this, toucher };
+	VMValue params[2] = { (DObject*)item, toucher };
 	VMReturn ret[2];
 	int res;
 	AActor *tret;

src/g_inventory/a_pickups.h

@@ -73,8 +73,6 @@ public:
 	virtual void Serialize(FSerializer &arc) override;
 	virtual void Tick() override;
 
-	bool CallTryPickup(AActor *toucher, AActor **toucher_return = NULL);	// Wrapper for script function.
-
 	void DepleteOrDestroy ();			// virtual on the script side. 
 
 	bool DoRespawn();
@@ -98,4 +96,6 @@ public:
 	FSoundIDNoInit PickupSound;
 };
 
+bool CallTryPickup(AInventory *item, AActor *toucher, AActor **toucher_return = nullptr);
+
 #endif //__A_PICKUPS_H__

src/p_actionfunctions.cpp

@@ -4150,7 +4150,7 @@ static bool DoRadiusGive(AActor *self, AActor *thing, PClassActor *item, int amo
 			}
 			gift->flags |= MF_DROPPED;
 			gift->ClearCounters();
-			if (!gift->CallTryPickup(thing))
+			if (!CallTryPickup(gift, thing))
 			{
 				gift->Destroy();
 				return false;

src/p_conversation.cpp

@@ -972,7 +972,7 @@ static void HandleReply(player_t *player, bool isconsole, int nodenum, int reply
 					item->IntVar(NAME_AmmoGive1) = 40;
 				}
 				item->flags |= MF_DROPPED;
-				if (!item->CallTryPickup(player->mo))
+				if (!CallTryPickup(item, player->mo))
 				{
 					item->Destroy();
 					takestuff = false;

src/p_mobj.cpp

@@ -926,7 +926,7 @@ AInventory *AActor::GiveInventoryType (PClassActor *type)
 	if (type != NULL)
 	{
 		item = static_cast<AInventory *>(Spawn (type));
-		if (!item->CallTryPickup (this))
+		if (!CallTryPickup (item, this))
 		{
 			item->Destroy ();
 			return NULL;

src/p_user.cpp

@@ -980,7 +980,7 @@ void APlayerPawn::GiveDeathmatchInventory()
 			if (key->special1 != 0)
 			{
 				key = (AInventory*)Spawn(PClassActor::AllActorClasses[i]);
-				if (!key->CallTryPickup (this))
+				if (!CallTryPickup (key, this))
 				{
 					key->Destroy ();
 				}