AMXX.pl: Support AMX Mod X i SourceMod

To już chyba błąd autora, bo ja tego nie zmieniałem. Po włączeniu Assassin Mode na więcej niż ~10 osób, w logach jest błąd w którym plugin wysyła wiadomość do id 0.

Na Nemesis Mode nie występuje ten błąd, bo wogóle nie ma message ScreenFade i ScreenShake.

L 08/04/2018 - 18:29:25: Plugin called message_begin with an invalid message id (0).
L 08/04/2018 - 18:29:25: [AMXX] Displaying debug trace (plugin "zp50_gamemode_assassin.amxx")
L 08/04/2018 - 18:29:25: [AMXX] Run time error 10: native error (native "message_begin")
L 08/04/2018 - 18:29:25: [AMXX]    [0] zp50_gamemode_assassin.sma::zp_fw_gamemodes_start (line 164)

zp50_gamemode_assassin.amxx:

public zp_fw_gamemodes_start()
{	
	// Turn player into survivor
	zp_class_assassin_set(g_TargetPlayer)
	
	// Remaining players should be humans (CTs)
	new id
	for (id = 1; id <= g_MaxPlayers; id++)
	{
		// Not alive
		if (!is_user_alive(id))
			continue;
		
		// This is our assassin
		if (zp_class_assassin_get(id))
			continue;
		
		// Switch to CT
		cs_set_player_team(id, CS_TEAM_CT)
	
		// Make a screen fade 
		message_begin(MSG_ONE, g_msgScreenFade, _, id) // line 164
		write_short(UNIT_SECOND*5) // duration
		write_short(0) // hold time
		write_short(FFADE_IN) // fade type
		write_byte(250) // red
		write_byte(0) // green
		write_byte(0) // blue
		write_byte(255) // alpha
		message_end()
			
		// Make a screen shake [Make it horrorful]
		message_begin(MSG_ONE_UNRELIABLE, g_msgScreenShake, _, id)
		write_short(UNIT_SECOND*75) // amplitude
		write_short(UNIT_SECOND*7) // duration
		write_short(UNIT_SECOND*75) // frequency
		message_end()
	}
	
	// Play assassin sound
	if (get_pcvar_num(cvar_assassin_sounds))
	{
		new sound[SOUND_MAX_LENGTH]
		ArrayGetString(g_sound_assassin, random_num(0, ArraySize(g_sound_assassin) - 1), sound, charsmax(sound))
		PlaySoundToClients(sound)
	}
	
	if (get_pcvar_num(cvar_assassin_show_hud))
	{
		// Show assassin HUD notice
		new name[32]
		get_user_name(g_TargetPlayer, name, charsmax(name))
		set_hudmessage(HUD_EVENT_R, HUD_EVENT_G, HUD_EVENT_B, HUD_EVENT_X, HUD_EVENT_Y, 1, 0.0, 5.0, 1.0, 1.0, -1)
		ShowSyncHudMsg(0, g_HudSync, "%L", LANG_PLAYER, "NOTICE_ASSASSIN", name)
	}

	// Setting Dark Lightning For The Mode
	get_cvar_string ( "zp_lighting", g_LValue , 2 )
	if ( !equali( "g_LValue" , "a" ) )
	{

		set_cvar_string ( "zp_lighting" , "a" )

	}
}

Dodam, że tryby testowałem na XFakePlayers (bo podboty mózgu nie mają >.<).

W sumie to może być też wina botów bo co runde bez powodu ginie ich powoła.

Można jakoś naprawić ten błąd ?

Pokaż całe sma, a jeśli to silnik to pokaż deklarację;

g_msgScreenFade

ZP 5.0 to raczej kilkanaście - kilkadziesiąt mini-silników, nie widziałem jeszcze tej wersji z jednym dużym silnikiem, jest w ogóle taki ?

Cały plik zp50_gamemode_assassin.sma:

 zp50_gamemode_assassin.sma   6,28 KB   76 Ilość pobrań

g_msgScreenShake = get_user_msgid("ScreenShake")
g_msgScreenFade = get_user_msgid("ScreenFade")

Nie powinno to być w init? Nie jestem pewny, czy zrobi to jakąkolwiek różnicę, ale według starego zp 4.3 jest to w init i warto spróbować. Jak nie zadziała, to poszukamy przyczyny gdzie indziej.

Dodałem to do init i nic to nie zmieniło

Sugerując się silnikiem 4.3 spróbuj zamienić pierwszy argument message_begin na:

MSG_ONE_UNRELIABLE

Podobno jest to "zawodna" metoda, ale warto spróbować. Od dłuższego czasu grzebie się z silnikiem 4.3 i nie miałem nigdy problemu z tym konkretnym errorem.

Sama rejestracja ScreenFade jest raczej poprawna - niezbyt złożona (zaledwie z deklaracji handlera, przypisania indexu wiadomości i użycia w message_begin), więc też mało prawdopodobne, że jest to problemem.

Już tego próbowałem, nic to nie dało

Sprawdź, potem dołącz plik z logiem z linii 216.

Dalej to samo (chociaż to już chyba ten drugi message ?)

L 08/05/2018 - 12:36:19: gmsgScreenFade message id = 98 (Playerid: 2)
L 08/05/2018 - 12:36:19: Plugin called message_begin with an invalid message id (0).
L 08/05/2018 - 12:36:19: [AMXX] Displaying debug trace (plugin "zp50_gamemode_assassin.amxx")
L 08/05/2018 - 12:36:19: [AMXX] Run time error 10: native error (native "message_begin")
L 08/05/2018 - 12:36:19: [AMXX]    [0] zp50_gamemode_assassin.sma::zp_fw_gamemodes_start (line 166)
L 08/05/2018 - 12:36:19: Log file started (file "cstrike\addons\amxmodx\logs\ZombiePlague_20180805.log") (game "cstrike") (amx "1.8.2")

@edit aSiorr, zrobiłem tak samo z ScreenShake i już działa, nie bardzo wiem co to za różnica ale poprawione

message_begin( MSG_ONE, gmsgScreenFade, {0,0,0}, id );

a

message_begin(MSG_ONE, g_msgScreenFade, _, id);

Przekazanie "_" czyli pominięcie argumentu powinno zostawić przypisaną do zmiennej wartość początkową:

float vecOrigin[3];

Jak na moje warunek pokazany niżej łapie się na tym, że vecOrigin ma wartości < 1, ale to może być tylko moje krzywe myślenie na ten moment.

Dokumentacja message_begin mówi, że argument ten:

origin - położenie źródła wiadomości, nie zawsze potrzebne.

Nie zawsze, ale ewidentnie czasami wymagane - jak widać po wcześniejszych postach.

Według source-code "messages":

if (params[2] < 1 || ((params[2] > 63) && !GET_USER_MSG_NAME(PLID, params[2], NULL))) // maximal number of engine messages 
{
	LogError(amx, AMX_ERR_NATIVE, "Plugin called message_begin with an invalid message id (%d).", params[2]);
	return 0;
}

"maximal number of engine messages" więc jest możliwość (jeśli dobrze interpretuje komentarz), że rzeczywiście zawołane zostało zbyt dużo "engine messages".

Wypadałoby wyprintować origin podawany do funkcji message_begin, żeby sprawdzić jaką rzeczywiście przyjmuje wartość - jeśli > 1 to nie przeszła druga część ifa, czyli zbyt dużo wiadomości silnika. Nie mam teraz czasu, żeby zagłębiać się w to, ale skoro problem rozwiązany to chyba nie ma co szukać dziury w całym.