[Joe_]

Code:

#define white 0xFFFFFFFF

new BadCheckTimer;

new BadWeapons[3] =
{
	38, 36, 39
};

forward MessageToAdmins(color,const string[]);

Code:

BadCheckTimer = SetTimer("BadCheck",5000,1);

Code:

KillTimer(BadCheckTimer);

Code:

forward BadCheck();
public BadCheck()
{
  new weap, string[128], pname[MAX_PLAYER_NAME]; // Shortened by 128 Cells, 256 Cells are crap kthx.
	for(new i, b = GetMaxPlayers(); i < b; i++) // Loop around MaxPlayers Server.cfg varible & removed IsPlayerConnected
	{
	  GetPlayerName(i, pname, sizeof(pname));
	  
	  weap = GetPlayerWeapon(i);
	  for(new w; w < sizeof(BadWeapons); w++)
	  {
			if(weap == BadWeapons[w])
			{
				format(string,sizeof(string),"[ ! ]%s Has got a %s.", pname, weap);
				MessageToAdmins(white,string);
				ResetPlayerWeapons(i);
			}
			return 1;
		}
	}
	return 1;
}

public MessageToAdmins(color,const string[])
{
	for(new a,b=GetMaxPlayers();a<b;a++)
	{
		if(IsPlayerAdmin(a)) SendClientMessage(a, color, string);
	}
	return 1;
}

Made quickly so you can see how much of your code is undeeded (This probably has some missing code, but It -shows- what can be shortened)

Don't even bother wasting your time to say it won't work, read above