GameEventListener2 callback in list that should NOT be - player_death!

Ajaxx · **Author**

I’m try to display messages to users when they kill someone or die. I’m adding a listener like so

Code:

gameevents->AddListener(this, "player_death", true);

When I test my plugin I don’t see the message and I see this in the server console

Quote:

GameEventListener2 callback in list that should NOT be - player_death!

Does anyone have any suggestions?

psychonic

Quote:

Originally Posted by psychonic @ MM:S Dev Wiki

Code:

[23:49] <psychon1c> oh, for events, IEventListener2 has an extra member, starting in swarm i think
[23:49] <MaxLap> good thing i wasn't aiming at being ready for release then!
[23:49] <psychon1c> GetDebugId or GetEventDebugId, something like that
[23:50] <psychon1c> and just return 42 in the impl. there's a define for it, but i don't remember the name
[23:50] <psychon1c> should be in the header

https://forums.alliedmods.net/showthread.php?t=194338

Ajaxx

I have the latest HL2SDKCSGO and MMSOURCE19. When I implement GetEventDebugID() returning NULL like so

Code:

int	 StubPlugin::GetEventDebugID()
{
	return NULL;
}

I get the message

Quote:

GameEventListener2 callback in list that should NOT be - player_death!

When I implement GetEventDebugID() returning 42 like you suggest

Code:

int	 StubPlugin::GetEventDebugID()
{
	return 42;
}

The server crashes when the event is fired.

psychonic

Quote:

Originally Posted by Ajaxx

The server crashes when the event is fired.

Well, debug it.

It sounds like your event handler wasn't firing before having the correct debug id set and it is firing now, and likely crashing inside of it.

Ajaxx

You’re right, the event is firing now and has uncovered an issue elsewhere in my code. It seems to be throwing an exception here:

Code:

const char* StubPlugin::GetSteamIdOfUserId(int userId)
{
	#undef GetClassName

	edict_t* entity;

	for  (int i = 1; i < MAX_PLAYERS; i++)
	{
		entity = (edict_t *)(gpGlobals->pEdicts + i); <<<<<< exception thrown here

		if (entity && !entity->IsFree())
		{
			if (FStrEq(entity->GetClassName(), "player"))
			{
				if (engine->GetPlayerUserId(entity) == userId)
				{
					return engine->GetPlayerNetworkIDString(entity);
				}
			}
		}
	}

	#define GetClassName  GetClassNameA

	return NULL;
}

It might be helpful to note I’m testing this with bots.

psychonic

Have you confirmed that gpGlobals was properly populated? Also, what is MAX_PLAYERS set to?

Ajaxx

Right again, thanks psychonic I have my messages working now!

Powerlord

Suggestion: Use gpGlobals->maxClients instead of MAX_PLAYERS.