[BigETI]

Quote:

Originally Posted by ****** You REALLY need to look up ALS if you want this to be compatible with other libraries. Currently this will give warnings when used with YSI/ZCMD/Streamer/foreach/many other less common ones.

Okay didn't thought about the ALS thing.

Quote:

Originally Posted by ****** Also, this only seems to work inside one script, which frankly isn't all that useful - if you change a skin in your mode you know it has changed, knowing when other scripts (e.g. filterscripts) change the skin is much more useful information.

The point is, that this should get only called in scripts you need the callbacks else if I use CallRemoteFunction instead of CallLocalFunction and I include this inside several scripts, so a callback will get called on how much scripts I have included this include.

Quote:

Originally Posted by ****** I was looking through the code and it seemed quite nice, until I saw "SetVar" - why? That only obfuscates a common PAWN ability, making code harder to read.

Well you can comment #define USE_PVAR_CALLBACKS to disable the OnPVar Callbacks.

Quote:

Originally Posted by ****** Edit: I should add that I'm one of the people who criticise problems with things and assume that everything not mentioned (especially effort) is fine, but I should learn to praise more as it helps some people/

I may point that your reviews just helps peoples to improve their own works

Btw I just added

pawn Код:

public OnPlayerSpecialActionChange(playerid, oldsaction, newsaction)

since I had forget it on the last release and added some descriptions on the main post.

Of course I'll use the ALS technique on the next release since as you can see that ****** wrote that my include doesn't fit with another certain used includes.

Anyways thank you all.