[Chanz]

I've never worked with ZR, but if you don't want to give a zombie a weapon, then check if the client is a zombie and don't give it to him.

It would be bad to add an automatic check only for ZR, imagine we need to do this for as example Hide and Seek and every other mod/plugin etc. even if its only for Client_GiveWeapon, its performance would drop extreamly to check these, not to mention the work to keep everything working.

[berni]

Originally Posted by zeroibis For example you can use this command to give a zombie weapons in a ZR server.

It's a Function, not a Command.
I don't know much about ZR either, but the Scripter is responsible himself who he calls this function on,
there are maybe coders who do want to give weapons to zombies, implementing a restriction would be a bad idea.

[zeroibis]

Oh I am not requesting it be made to work like that I am only stating that functionally it does not work the same as GivePlayerItem() and that programers should be careful which one they use depending on the situation.

[Chanz]

well every author needs to be careful even with GivePlayerItem() since you can't just hope that a function is blocked by another plugin. you should always be sure and minimize function calls by filtering out unwanted clients etc.

[zeroibis]

Question: is Math_GetRandomInt(min, max); inclusive?

In that are min and max possible outputs for this function or only values between them. Same for the Math_GetRamdomFloat...

[berni]

Originally Posted by zeroibis Question: is Math_GetRandomInt(min, max); inclusive? In that are min and max possible outputs for this function or only values between them. Same for the Math_GetRamdomFloat...

yes, it is inclusive, min and max are also part of the range.

Another note on GivePlayerItem(), this function is not very safe, as it spawns
a weapon or an item at the players position and hopes the player touches it.
If the player is currently inside a wall or does already have that item, the item would be dropped to the floor.

[zeroibis]

Cool thanks!

Also yes I know of the GivePlayerItem() issue which is why I strip weapons or check to see if they have what I am giving before use.

[berni]

New smlib update available: smlib 0.9.6 beta
Important Note: SVN folder structure has been changed.

Compatibility breaks: yes

Changes:

Added Client button stock functions. Created a plugin file smlib_test to compile all functions at once. Fixed a shitload of compile errors / warnings. Added Function Client_IsLookingAtWall() Added Functions Client_GetClass() and Client_SetClass() Corrected some function descriptions Fixed Function Entity_HasSpawnFlags() (missing ()), thanks naris

Download here.

[FeuerSturm]

Berni, you might want to take a look at this stock in clients.inc:

/**
 * Gets the name of the last place (if set by the game)
 *
 * @param entity			Entity index.
 * @param buffer			String buffer
 * @param size				Size of the String buffer
 * @noreturn
 */
stock Client_GetLastPlaceName(client, String:buffer[], size)
{
	SetEntPropString(client, Prop_Send, "m_szLastPlaceName", buffer);
}

[berni]

ok "S" should be "G" commited to svn.