Quote:
Originally Posted by Silvers
Maybe call "IsValidEdict" and use "GetEdictClassname".
Also you should check "channel == SNDCHAN_STATIC" before all the other checks since that consumes the least processing power while the others require more and if the channel is different then you just wasted a bunch of CPU time. I would personally also init the variable as "static char className" so it's not constantly created over and over, sound hooks fire very often.
|
The topic is about
IsValidEntity() not being enough of a check in order to use some of the other functions, but you started it and I will say that stack allocation of arrays should be dirt cheap, no? If it's
static, then the plugin will take more runtime space, no?
Edit: Ok, i see this in the sourcepawn transition syntax docs:
Also note, there is no equivalent of decl in the new declarator syntax. decl is considered to be dangerous and unnecessary. If an array's zero initialization is too costly, consider making it static or global. .
Maybe
decl should have been kept in some way.
__________________