[Dennis12]

Old code ..

1. Why you loop all players?

Code:

for(new i = 0; i <MAX_PLAYERS; i++)

If you don't want to use ITERATORS ( YSI\y_iterate ) to select only from VIP LIST you can make something like this, but with iterators is much better.

Code:

for(new i = 0, j = GetPlayerPoolSize(); i <= j; i++)

This loop will go up to the highest ID on the server, the loop you have with MAX_PLAYERS goes through the 1000 players (define default, even if you have 100 on the server).

Why INTEGER only for 0/1 (false / true) ..? Use the boolean variable .. pGOD should be a bool.

Code:

bool: pGOD ( .. in pInfo )

At least if you used a switch and not if / else at CMD: vgod ..


EDIT: OHHH, SORRY MAN, IT'S YOUR FIRST SCRIPT ..
Take my words above as some tips, not critics, now I just saw that it is your first script, very good for a first script, you are on a good road.